Skip to content

Conversation

@stanislav-tkach
Copy link
Member

Description

  • Update the rbac-registration dependency to 0.0.7 version.

@stanislav-tkach stanislav-tkach self-assigned this Aug 10, 2025
@stanislav-tkach stanislav-tkach added review me PR is ready for review rr: gatekeepers Required review by gatekeepers team labels Aug 10, 2025
@github-actions
Copy link
Contributor

📚 Docs Preview

The docs for this PR can be previewed at the following URL:

https://docs.dev.projectcatalyst.io/voices/rbac-update-0-0-7

@Mr-Leshiy
Copy link
Contributor

@stanislav-tkach if its not urgent I would also wait for this fix to be integrated to the cat-gateway input-output-hk/catalyst-libs#467

@Mr-Leshiy Mr-Leshiy added the do not merge yet PR is not ready to be merged yet label Aug 11, 2025
@stanislav-tkach
Copy link
Member Author

@Mr-Leshiy I guess it isn't urgent, but I'm not sure I understand the reasoning. Why do we want to have one larger update instead of two smaller ones?.. Does it really matter? It isn't that we have something like continuous deployment.

@Mr-Leshiy
Copy link
Contributor

@stanislav-tkach it would be just a versioning bump, so no any other modifications required, but dont have a strict position here. and as CI is already passed for this PR you are right lets merge it already

@Mr-Leshiy Mr-Leshiy removed the do not merge yet PR is not ready to be merged yet label Aug 11, 2025
Copy link
Contributor

@Mr-Leshiy Mr-Leshiy left a comment

Choose a reason for hiding this comment

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

LGTM

@Mr-Leshiy Mr-Leshiy merged commit f9f07ab into main Aug 11, 2025
48 checks passed
@Mr-Leshiy Mr-Leshiy deleted the rbac-update-0-0-7 branch August 11, 2025 07:42
@github-project-automation github-project-automation bot moved this from New to ✅ Done in Catalyst Aug 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review me PR is ready for review rr: gatekeepers Required review by gatekeepers team

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants