ocaml-opam / opam-depext

Attempt to automate the installation of system packages required by OPAM packages
Other
33 stars 28 forks source link

Make `2.0` the default branch #122

Closed carlosdagos closed 3 years ago

carlosdagos commented 4 years ago

Hello,

I think it'd be nice to make 2.0 the default repository branch, since it makes finding the effort being done in https://github.com/ocaml/opam/pull/3975 a bit easier.

With that said, is this repository still accepting PRs? At the time of writing https://github.com/ocaml/opam/pull/3975 isn't merged and I'd like to discuss a potential change to opam-depext (which could make its way onto https://github.com/ocaml/opam/pull/3975).

Thank you :smile:

XVilka commented 4 years ago

Or merge it in the master like the similar situation in opam-publish: https://github.com/ocaml/opam-publish/pull/85

rjbou commented 4 years ago

Agree on merging/setting 2.0 default branch, it's less confusing, plus the notice is only in e 2.0 branch readme.

The repo is not yet frozen (because the depext PR is still open), but we won't update the plugin (unless critical bug fix for ex.).

We can discuss on these changes in the dedicated opam issue

dra27 commented 3 years ago

Default branch has been 2.0 for a while - at this stage it's probably fine just to leave the repo as somewhat strangely having two branches?!

rjbou commented 3 years ago

Agree!