search

tarides / dune-release

Streamlining the release of dune packages to opam
ISC License
114 stars 38 forks source link

`dune-release` gets confused about a documentation link pointing to `index.html` #464

Open lukstafi opened 1 year ago

lukstafi commented 1 year ago

With the check command, I was getting: opam field doc cannot be parsed by dune-release. The field was this: doc: "https://lukstafi.github.io/ppx_minidebug/ppx_minidebug/index.html" With the actual release, I was getting: cp: cannot create directory 'ppx_minidebug/index.html': No such file or directory Problem went away when I removed the /index.html suffix in the documentation stanza of dune-project.

Leonidas-from-XIV commented 1 year ago

Hmm, yes indeed, check probably error out with a better error. That's a bug for sure.

Want to submit a PR to fix it? It should be reasonably easy.