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.

Sign up for GitHub

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

Jump to bottom

Update security checks #509

Merged
merged 2 commits into from
May 23, 2024
Merged

Update security checks #509

merged 2 commits into from
May 23, 2024

Conversation

Zeitsperre
Copy link
Contributor

Overview

  • Added custom endpoints for security in main.yml
  • Now using commit hashes for specifying action versions

Additional Information

I think raven will be ready for release after this PR

@Zeitsperre Zeitsperre requested review from huard and tlvu May 21, 2024 18:40
@Zeitsperre Zeitsperre self-assigned this May 21, 2024
tlvu
tlvu approved these changes May 23, 2024
View reviewed changes
Copy link
Contributor

@tlvu tlvu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me but please also ensure tuto notebook autodeploy to PAVICS and nightly Jenkins are not broken with this change. Take a look at my comment on Finch repo for the same default branch name change.

@Zeitsperre Zeitsperre merged commit e79fbc2 into main May 23, 2024
11 checks passed
@Zeitsperre Zeitsperre deleted the update-actions branch May 23, 2024 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tlvu tlvu tlvu approved these changes

@huard huard Awaiting requested review from huard

Assignees

@Zeitsperre Zeitsperre

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Zeitsperre @tlvu