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

Update provider #26632

Merged
merged 2 commits into from
Sep 29, 2024
Merged

Update provider #26632

merged 2 commits into from
Sep 29, 2024

Conversation

mbarbin
Copy link
Contributor

@mbarbin mbarbin commented Sep 27, 2024

  • Add maintainer email to existing provider packages

  • Flagged older provider.0.0.4 as deprecated.

This version has a bug in the binding lookup I'd be happy to forget. opam.ocaml.org says this package is not used so I propose to simply delete this release. Is that technically acceptable?

@avsm
Copy link
Member

avsm commented Sep 28, 2024

We can't remove packages, unfortunately -- there may be other opam repositories that depend on this ourside of ours. However, you can add a messages to indicate it is deprecated...

@mbarbin
Copy link
Contributor Author

mbarbin commented Sep 28, 2024

OK thanks for letting me know. I looked at the manual and discovered a wealth of options I wasn't aware of, such as ... a deprecated flag for releases with known bugs! Thanks, updated 😃

@mbarbin mbarbin marked this pull request as ready for review September 28, 2024 20:46
@avsm
Copy link
Member

avsm commented Sep 29, 2024

Looks perfect, thanks for the flexibility!

@avsm avsm merged commit a76faec into ocaml:master Sep 29, 2024
3 checks passed
@mbarbin mbarbin deleted the update-provider branch September 29, 2024 14:40
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