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

Fix ---V to be -V #388

Merged
merged 2 commits into from
Sep 9, 2021
Merged

Conversation

Leonidas-from-XIV
Copy link
Member

@Leonidas-from-XIV Leonidas-from-XIV commented Sep 9, 2021

Cmdliner accepts everything that is longer than one character as a long option, hence it made -V into ---V long option.

This has been broken since -V was added in 809f539, so maybe nobody uses it and it might sense to just remove that option?

Cmdliner accepts everything that is longer than one character as a long
option, hence it made `-V` into `---V` long option.
@NathanReb
Copy link
Contributor

The --pkg-version ... option worked all along so I think we can keep it around. It's just the "short" option that was broken.

@NathanReb NathanReb merged commit f3f38cc into tarides:main Sep 9, 2021
@Leonidas-from-XIV Leonidas-from-XIV deleted the fix-triple-dash-v branch September 9, 2021 14:24
Leonidas-from-XIV added a commit to Leonidas-from-XIV/opam-repository that referenced this pull request Oct 8, 2021
CHANGES:

### Added

- Added support for creating releases from unannotated Git tags. `dune-release`
  supported unannotated tags in a few places already, now it supports using
  them for creating a release. (tarides/dune-release#383, @Leonidas-from-XIV)

### Fixed

- Change the `---V` command option to be `-V` (tarides/dune-release#388, @Leonidas-from-XIV)
- Infer release versions are inferred from VCS tags. This change allows using
  `dune-release` on projects that do not use the changelog or have it in a
  different format.  (tarides/dune-release#381, tarides/dune-release#383 @Leonidas-from-XIV)
- Fix a bug where `dune-release` couldn't retrieve a release on GitHub if the
  tag and project version don't match (e.g. `v1.0` vs `1.0`). `dune-release`
  would in such case believe the release doesn't exist, attempt to create it
  and subsequently fail. (tarides/dune-release#387, tarides/dune-release#395, @Leonidas-from-XIV)
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