Skip to content

[REQUEST] Integrate CI Against New PR #6

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

Open
3 tasks done
bhavik2936 opened this issue Oct 15, 2022 · 0 comments
Open
3 tasks done

[REQUEST] Integrate CI Against New PR #6

bhavik2936 opened this issue Oct 15, 2022 · 0 comments
Labels
enhancement New feature or request

Comments

@bhavik2936
Copy link
Owner

Please make sure to follow & check below prerequisites.

  • I checked the documentation and didn't find this feature
  • I checked to make sure that this feature has not already been requested

Describe the Issue

On raising a PR for new/modifying existing docker-compose file, it is hard to check for minute error.

Expected Solution

On every PR raised, we would like to have CI checks to be integrated. Which verifies the docker-compose file syntactically (if feasible then by running).

Alternatives

If docker-compose validator does not seem to be working as CI check; We can opt-in for integrating it as (pre-push) git hooks.

And the last option to opt will be running the docker-compose file in CI check.

Additional Context

There already exists convert (previously config) command to parse the docker-compose file for target platform. Which serves the intention to validate the docker-compose file.

A set of pre-commit hooks are implemented on this repo.

A python script to validate docker-compose is available on this repo.

@bhavik2936 bhavik2936 added the enhancement New feature or request label Oct 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant