Skip to content

Add a CLI flag for prerelease#517

Merged
Leonidas-from-XIV merged 3 commits into
tarides:mainfrom
Sudha247:pre-release
Mar 25, 2026
Merged

Add a CLI flag for prerelease#517
Leonidas-from-XIV merged 3 commits into
tarides:mainfrom
Sudha247:pre-release

Conversation

@Sudha247
Copy link
Copy Markdown
Member

Fixes #516.

Adds a flag to mark a release as an aprerelease. Support for this already exists in the GitHub REST API. This patch exposes it to dune-release CLI.

Copy link
Copy Markdown
Member

@shonfeder shonfeder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me! (for whatever that is worth :) Thanks!)

Comment thread CHANGES.md Outdated
@Leonidas-from-XIV Leonidas-from-XIV self-requested a review March 24, 2026 14:01
Copy link
Copy Markdown
Member

@Leonidas-from-XIV Leonidas-from-XIV left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small improvements possible but looks good from my side.

(That said this is not surprising, this PR is the result of @Sudha247 and me pairing to add this feature)

Comment thread bin/cli.ml Outdated
Comment thread tests/bin/prerelease/run.t Outdated
Comment thread tests/lib/test_github_v3_api.ml Outdated
Fixes tarides#516. Adds a flag for marking a release as aprerelease. The support for this already exists in the GitHub REST API calls. This patch exposes it to dune-release CLI.

Signed-off-by: Sudha Parimala <sudharg247@gmail.com>
Deduplicate code in tests and minor clean-up

Signed-off-by: Sudha Parimala <sudharg247@gmail.com>
@Sudha247
Copy link
Copy Markdown
Member Author

Thanks for the reviews @shonfeder and @Leonidas-from-XIV! I've addressed your feedback and updated the PR accordingly.

Comment thread CHANGES.md Outdated
Remove the 'Closes' part as this is not being recorded in the Changelog.
@Leonidas-from-XIV Leonidas-from-XIV merged commit 54ee06a into tarides:main Mar 25, 2026
3 checks passed
Leonidas-from-XIV added a commit to Leonidas-from-XIV/opam-repository that referenced this pull request Mar 26, 2026
CHANGES:

### Added

- Add support for marking a release as a prerelease. (tarides/dune-release#517, @Sudha247)
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.

Support for publishing GitHub releases as "pre-release"

4 participants