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

sbojanic/ci improve security #60

Merged
merged 1 commit into from
Nov 27, 2024
Merged

sbojanic/ci improve security #60

merged 1 commit into from
Nov 27, 2024

Conversation

sale3
Copy link
Contributor

@sale3 sale3 commented Nov 25, 2024

Description (e.g. "Related to ...", etc.)

ci: replace tags with commit hashes

Code review checklist (for code reviewer to complete)

  • Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
  • Title summarizes what is changing
  • Commit messages are meaningful (see this for details)
  • Tests have been included and/or updated, as appropriate
  • Docstrings have been included and/or updated, as appropriate
  • Standalone docs have been updated accordingly

Automated tests, benchmarks and linters

You can run the tests, lints and benchmarks that are run on CI locally with:

just ci

@sale3 sale3 requested a review from n-dusan November 25, 2024 19:43
@sale3 sale3 self-assigned this Nov 25, 2024
@n-dusan n-dusan merged commit e8c3996 into main Nov 27, 2024
4 checks passed
@n-dusan n-dusan deleted the sbojanic/ci-improve-security branch November 27, 2024 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@n-dusan n-dusan n-dusan approved these changes

Assignees

@sale3 sale3

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sale3 @n-dusan