-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
bump(main/mdbook) :v0.5 #27160
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
bump(main/mdbook) :v0.5 #27160
Conversation
TomJo2000
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are we updating this package to a beta release anyway?
We don't ship beta for a lot of packages, so why for mdbook?
packages/mdbook/build.sh
Outdated
| TERMUX_PKG_MAINTAINER="@termux" | ||
| TERMUX_PKG_VERSION="0.4.52" | ||
| TERMUX_PKG_REVISION=1 | ||
| TERMUX_PKG_VERSION="0.5.0-beta.2" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- As mentioned before is both bump(root/nexttrace): v1.5.0 #27157 (comment) and bump(root/nexttrace): v1.5.0 #27158 (comment)
This is gonna need a~instead of a-.
You'll also need to fix up the variable expansion in the SRCURL when doing that.
as previously stated in bump(root/nexttrace): v1.5.0 #27157 (comment)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
well i had initially added the ~ but forgot to update the variable expansion so that happed and for the y are we shipping beta updates cus idk random motivation ig
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No other package of mdbook is shipping 0.5.0-beta.2.
Well besides crates.io, but that's the registry for rust packages.
https://repology.org/project/mdbook/versions
So I don't think there is a clear case for why we should be shipping it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh i see
|
Please double check your commit message |
|
Superseded by #27283 |
No description provided.