search

ocaml-opam / Camelus

Bot posting reports on opam-repository pull-request using a web hook
Other
18 stars 9 forks source link

Broken packages are not correctly detected #38

Open kit-ty-kate opened 4 years ago

kit-ty-kate commented 4 years ago

Camelus uses the branch from where PR are made from without merging it in master first when it checks for broken packages. Which for some people that don't use opam-repository that much can be a big problem.

For example this PR has one commit on top of a commit made in 2017: https://github.com/ocaml/opam-repository/pull/16158. This example is not the first one I've encountered, it's the first time I bother opening an issue about it. There are plenty of similar examples to take from.

thomasblanc commented 4 years ago

@AltGr I guess you can assign me to this :-)