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

Update Github Actions file #80

Merged
merged 1 commit into from
May 10, 2024

Conversation

yuki-was-taken
Copy link
Contributor

Is your pull request linked to an existing issue?

#77

What is your pull request about?:

I changed the following things:

  1. Updated actions/checkout to v4. (it was v3 previously)
    2.Renamed the file from build-test to test-catnip, since it isn't just about building Catnip anymore.
  2. Changed jiro4989/setup-nim-action@v1 to setup the stable version of Nim, as well as use the secret Github token.
  3. Changed nim release to nim debug to give further information if the test ever fails at that stage.

Any other disclosures/notices/things to add?

The ${{ secrets.GITHUB_TOKEN }} is created when the Github Action thing was set up, and I'm sure that no one has access to it, (except for the runner, of course) so it's safe to include.

See https://docs.github.com/en/actions/security-guides/automatic-token-authentication#about-the-github_token-secret

@yuki-was-taken
Copy link
Contributor Author

Also the pull request template works now!

@iinsertNameHere
Copy link
Owner

Also the pull request template works now!

Looks very good! Thank you

@iinsertNameHere iinsertNameHere merged commit ad9e8ba into iinsertNameHere:main May 10, 2024
1 check passed
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