-
Notifications
You must be signed in to change notification settings - Fork 140
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
v1.0.0 Release Tracking Issue #203
Comments
Should we really release 1.0? I feel like there's a lot of functionality that's relatively easy to implement before, and being 1.0 kind of suggests stability. What about this plan:
|
I have a bizarre case that would benefit greatly from a 1.0: #200. It would also benefit greatly if the go-for-1.0 development cycle goes rather efficiently and smoothly. |
Huh that's interesting. I'm not familiar with the epoch, is it something that will always stay there and be annoying? Can it be a temporary thing? |
Once it's there it's there. It can't go away even if you (request to) remove it and package again, because it's a part of the version, and versions, obviously, go up monotonically in the archive. |
In that case I agree with releasing 1.0 as soon as possible. This being a binary means that as long as changes are backwards compatible any change is non breaking. What's left to do? #170 seems like a wontfix |
I've gone ahead and updated the top comment on the two remaining blockers. They're #44 which I want to get fixed before release since it's a breaking change in what we expect for replacement strings, and then bundling the contents of Currently my plan is:
|
Getting #44 sorted out took a bit longer than originally expected, but that's the last blocker done, so we're finally ready to try cutting a release. That being said it is holiday weekend for me (and I love the spooky season), so I don't expect too much to get done before next week 👻 🎃 |
Looks like all of the release assets were built and uploaded fine https://github.com/chmln/sd/releases/tag/1.0.0-pre-alpha.5 Seems like a good time for a checkin. Hi @chmln! Things are finally mostly wrapped up for a new release (aside from the task list above). Feel free to make sure that everything looks up to your standards Assuming that everything is fine, we can probably push out a release next week. I don't have publishing permissions for crates.io, but I think we can skip publishing any of the pre-release versions there |
Everything looks good @CosmicHorrorDev, excellent work! Sidenote: I also encourage a v1 release since the core functionality seems unlikely to undergo breaking changes |
version 0.8 is just as fine for Debian as 1.0 is. |
Thanks @chmln! I just updated my email for crates to be CosmicHorrorDev@pm.me (same as the one on my GitHub profile now) @nc7s can we get confirmation about potential issues, or lack there of, with using v0.8 instead of v1.0? |
@jbicha there's an old unrelated sd that went up to 0.74. I assume a 0.8 is still lower than that hence doesn't stop violating policy? @CosmicHorrorDev there's not really an (imminent) issue, the bug report is there for months. It's just me feeling not great about leaving a bug report open for so long ;) |
I forgot that version number sorting works differently. |
It would be weird to upgrade from 0.7 to 0.80 just because of the Debian needs. I think it's more manageable to go for the 1.0 especially since this is a binary and breaking changes are unlikely. If #194 Were to come about then we would need to be more careful but that's not yet the case. |
I haven't gotten around to reviewing that PR yet, but I don't view that as being an issue with having our next release be v1.0 only because I'm not planning on having |
Adding some extra context to the issues on the v1.0.0 milestone. After all of those issues are resolved we should be more-or-less prepared to cut the next major release
Overall there are a lot of duplicate issues in the tracker, but I want to keep the duplicates open for the time being since many of them are old, and there's no telling how many people are already subscribed to them
Prebuilt artifacts
Quite a few people want pre-built artifacts for different platforms. Some of these are already resolved and would be fixed by just publishing the next release. Overall we want
arm-unknown-linux-gnueabihf
, andaarch-unknown-linux-gnu
?Misc related issues
I also know that
aarch64-apple-darwin
was failing to test in #179. I've run into the same issue in another project, but the executable itself was fine. It just fails on a bad CPU type in the conditions that are used to build the executable for some reason 🤷Documentation
$
in the regex needs to be escaped as$$
. This should be documented in the help text and/or man page. We can probably look torg
for inspiration here--help
text needs to include everything that's in theman
page since it's common forman
pages to include more informationMisc changes
gen
in prebuilt releasesThe text was updated successfully, but these errors were encountered: