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

Validate RBI config in CI #228

Merged
merged 3 commits into from
Apr 29, 2024
Merged

Validate RBI config in CI #228

merged 3 commits into from
Apr 29, 2024

Conversation

andyw8
Copy link
Contributor

@andyw8 andyw8 commented Apr 29, 2024

To prevent issues such as #227

@andyw8 andyw8 requested a review from a team as a code owner April 29, 2024 12:30
@andyw8 andyw8 requested review from Morriar and egiurleo April 29, 2024 12:30
@andyw8 andyw8 added the chore label Apr 29, 2024
@andyw8 andyw8 requested a review from st0012 April 29, 2024 12:30
@andyw8 andyw8 force-pushed the andyw8/validate-rbi-config-in-ci branch from 43b10bd to 1bee3d8 Compare April 29, 2024 12:31
@andyw8 andyw8 force-pushed the andyw8/validate-rbi-config-in-ci branch from 1bee3d8 to 50c4e38 Compare April 29, 2024 12:31
@egiurleo
Copy link
Contributor

I think I'm a bit confused about the approach here -- why don't we need to verify the other configs in this repo?

@andyw8
Copy link
Contributor Author

andyw8 commented Apr 29, 2024

Good point, I've updated to verify config/default.yml too.

@andyw8 andyw8 merged commit 05cda77 into main Apr 29, 2024
13 checks passed
@andyw8 andyw8 deleted the andyw8/validate-rbi-config-in-ci branch April 29, 2024 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants