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

ci: generate the .opam files first #99

Merged
merged 1 commit into from
Dec 4, 2024
Merged

ci: generate the .opam files first #99

merged 1 commit into from
Dec 4, 2024

Conversation

liyishuai
Copy link
Member

@liyishuai liyishuai commented Dec 4, 2024

I've also disabled direct pushes to the master branch---now PR only.

@liyishuai liyishuai changed the title Revert a3e90caf8fe416000f8aff8e1b1e7ee3b309d589 Revert a3e90ca Dec 4, 2024
@liyishuai liyishuai changed the title Revert a3e90ca ci: keep the OPAM files Dec 4, 2024
@liyishuai liyishuai mentioned this pull request Dec 4, 2024
3 tasks
@liyishuai liyishuai changed the title ci: keep the OPAM files ci: generate the .opam files first Dec 4, 2024
@liyishuai liyishuai requested a review from fpottier December 4, 2024 07:59
Copy link
Collaborator

@fpottier fpottier left a comment

Choose a reason for hiding this comment

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

Looks good to me (though I am no CI expert).

@liyishuai liyishuai merged commit 33f30cd into master Dec 4, 2024
73 checks passed
@liyishuai liyishuai deleted the keep-opam branch December 4, 2024 12:41
@fpottier
Copy link
Collaborator

fpottier commented Dec 4, 2024

And I apologize for breaking the CI by removing the files *.opam from the repo.

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