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

chore: verify validity of built in rbac roles #13296

Merged
merged 1 commit into from
May 16, 2024

Conversation

Emyrk
Copy link
Member

@Emyrk Emyrk commented May 16, 2024

What this does

Verifies our built in roles are valid according to our policy.go. Working on custom roles requires the dynamic roles to adhere to these rules. Feels fair the built in ones do too.

Also some xerrors linting cleanup that somehow got missed before.

Copy link
Member Author

Emyrk commented May 16, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Emyrk and the rest of your teammates on Graphite Graphite

Copy link
Member

@johnstcn johnstcn left a comment

Choose a reason for hiding this comment

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

👍

coderd/rbac/roles.go Show resolved Hide resolved
coderd/rbac/roles.go Outdated Show resolved Hide resolved
coderd/rbac/roles.go Show resolved Hide resolved
coderd/rbac/roles.go Show resolved Hide resolved
Also some xerrors linting cleanup
@Emyrk Emyrk force-pushed the stevenmasley/custom_role_prep branch from 3b8ae70 to 65ec798 Compare May 16, 2024 15:30
@Emyrk Emyrk merged commit 194be12 into main May 16, 2024
32 checks passed
@Emyrk Emyrk deleted the stevenmasley/custom_role_prep branch May 16, 2024 17:07
@github-actions github-actions bot locked and limited conversation to collaborators May 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants