Skip to content

Conversation

@0xVolosnikov
Copy link
Contributor

What ❔

Why ❔

Is this a breaking change?

  • Yes
  • No

Checklist

  • PR title corresponds to the body of PR (we generate changelog entries from PRs).
  • Tests for the changes have been added / updated.
  • Documentation comments have been added / updated.
  • Code has been formatted.

Copy link
Member

@vladbochok vladbochok left a comment

Choose a reason for hiding this comment

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

Also we should set read only permissions

permissions: read-all

@0xVolosnikov
Copy link
Contributor Author

@vladbochok are you talking about pr-title.yml?

@vladbochok
Copy link
Member

@vladbochok are you talking about pr-title.yml?

All of them, including CI

@0xVolosnikov
Copy link
Contributor Author

Initially copied from zksync-era

@0xVolosnikov 0xVolosnikov requested a review from vladbochok May 27, 2025 21:26
@0xVolosnikov 0xVolosnikov enabled auto-merge (squash) May 27, 2025 21:44
@0xVolosnikov 0xVolosnikov merged commit e7e6e28 into main May 28, 2025
19 checks passed
@0xVolosnikov 0xVolosnikov deleted the vv-update-ci branch May 28, 2025 12: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