Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add github action to build on push #3

Open
wants to merge 1 commit into
base: Master
Choose a base branch
from

Conversation

cohensh
Copy link

@cohensh cohensh commented Feb 1, 2024

Enforce that the gradlew code builds for a PR

@ewpratten
Copy link
Member

ewpratten commented Feb 17, 2024

FYI, I arranged a special agreement with some GitHub staff back in 2018 that gives this specific organization some additional permissions that regular GitHub accounts don't have.

It appears that someone removed my permissions to view the billing settings, so I can't check the exact details.

My memory of the agreement is that we are allowed unlimited free organization members (usually costs $5/user/month), and we have some crazy high GitHub Actions cap that allows us to run much longer and more frequent builds that regular users can.

Basically, what I'm saying is, don't worry about sticking resource intensive operations in the CI pipeline. Our account should be able to handle it.

@awpratten
Copy link
Member

Moved ownership to Tiet where it should have always been. Students come and go. They should not be owners of this repo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants