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

Do test and build on all branches #4

Closed
wants to merge 1 commit into from
Closed

Conversation

fatboypunk
Copy link
Contributor

No description provided.

@fatboypunk
Copy link
Contributor Author

@dkln we can make it open source to use coveralls or I can remove ex_coveralls from the test setup.

@dkln
Copy link
Collaborator

dkln commented May 6, 2024

🤔 What would happen with our builds in that case if this repo becomes public? Are keys readable/visible etc.?

@fatboypunk
Copy link
Contributor Author

Which keys?

@dkln
Copy link
Collaborator

dkln commented May 6, 2024

The GITHUB_TOKEN

@fatboypunk
Copy link
Contributor Author

That is set in an env right? Not in code, so no that won't be public

@dkln
Copy link
Collaborator

dkln commented May 6, 2024

Ah right. Ok so maybe stupid question since I have no experience with public repo's and runners: if you fork the project and update the .github/ci.yml file, you couldn't do an echo ... on the command line and view it, right? (because that runner would be scoped on the env of the forker?)

@fatboypunk
Copy link
Contributor Author

I think this explains some https://securitylab.github.com/research/github-actions-preventing-pwn-requests/

@fatboypunk fatboypunk closed this Oct 25, 2024
@fatboypunk fatboypunk deleted the marcel/updates branch October 25, 2024 06:44
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.

2 participants