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

Add a test to ensure CircleCI configs are parsable #217

Closed
infraredgirl opened this issue Oct 12, 2021 · 2 comments
Closed

Add a test to ensure CircleCI configs are parsable #217

infraredgirl opened this issue Oct 12, 2021 · 2 comments
Assignees
Labels
enhancement New feature or request p:backlog

Comments

@infraredgirl
Copy link
Contributor

We've had a situation where a bad CircleCI config has broken the build, and it wasn't caught at PR merge time due to lack of tests. We should add some tests to ensure this does not happen again.

For more context, see #216 and #214.

@infraredgirl infraredgirl added enhancement New feature or request backlog labels Oct 12, 2021
@zackproser
Copy link
Contributor

The circle CI command line interface exposes circleci config validate <config-path> to make this simpler.

@james03160927
Copy link
Contributor

.circleCi/<>.yaml file is not visible inside the circle test pipeline. So unless someone runs this locally, it won't be possible to included this as part of the automated test pipeline.

Alternative is to duplicate the file into the test folder but it would be hard to keep both files updated at the same time and it will create potential data discrepancy, which will defeat the whole purpose of config testing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request p:backlog
Projects
None yet
Development

No branches or pull requests

3 participants