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

ci: code freeze through github actions #1588

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

joroshiba
Copy link
Member

@joroshiba joroshiba commented Sep 27, 2024

Summary

Adds functionality to block PRs into sections of code under code freeze within the CI workflow

Background

We are moving towards mainnet & undergoing audits. During past audits we have not had a firm mechanism to ensure we are not making changes to code under audit. This provides a stopper, but can be overriden still.

Changes

  • New CI workflow that checks for changes to code frozen pathways, fails and adds errors to those files as well as building an error summary. Using the override-freeze tag disables the check and changes reports to warnings.

Testing

Ran against this PR with multiple changes, tested both adding and removing the label to override.

@joroshiba joroshiba requested a review from a team as a code owner September 27, 2024 18:12
@github-actions github-actions bot added the ci issues that are related to ci and github workflows label Sep 27, 2024
@joroshiba joroshiba requested a review from a team as a code owner September 27, 2024 18:14
@github-actions github-actions bot added the sequencer-relayer pertaining to the astria-sequencer-relayer crate label Sep 27, 2024
@joroshiba joroshiba marked this pull request as draft September 27, 2024 18:14
@joroshiba joroshiba marked this pull request as ready for review September 27, 2024 19:30
Copy link

@WafflesVonMaple WafflesVonMaple left a comment

Choose a reason for hiding this comment

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

Very nice idea

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci issues that are related to ci and github workflows sequencer-relayer pertaining to the astria-sequencer-relayer crate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants