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

opam-ci-check: Command to list-revdeps in docker image #410

Merged
merged 1 commit into from
Jan 26, 2025

Conversation

punchagan
Copy link
Contributor

The opam-ci-check list command can be run locally to list revdeps for a package. It creates an isolated opam root for some isolation, but for a greater extent of isolation, this commit adds a list-revdeps command that can be used to list-revdeps locally using a Docker image.

@punchagan punchagan requested a review from shonfeder December 23, 2024 12:31
@punchagan
Copy link
Contributor Author

This PR is complementary to #403, and can be reviewed and merged independently of it.

@shonfeder shonfeder marked this pull request as ready for review January 26, 2025 01:38
The `opam-ci-check list` command can be run locally to list revdeps for
a package. It creates an isolated opam root for some isolation, but for
a greater extent of isolation, this commit adds a list-revdeps command
that can be used to list-revdeps locally using a Docker image.

This makes it easier to reproduce the CI runs more closely, when trying
to reproduce locally. And adds extra flexibility to be able to test on
different OSes etc.
@shonfeder shonfeder enabled auto-merge January 26, 2025 01:43
@shonfeder shonfeder merged commit e31e906 into ocurrent:master Jan 26, 2025
2 checks passed
@punchagan punchagan deleted the revdep-docker branch January 26, 2025 08:22
@punchagan
Copy link
Contributor Author

Thanks for taking care of this PR, @shonfeder !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants