Update the release script by adding some missing `git add`, which were
lost when switching to using Topiary from crates.io.
Doing so, we also remove the `--dry-run` step before publishing: the
usual workflow is to perform the dry run test, and if it succeeds,
immediately do the actual publication. This isn't very useful: if the
dry run fails, we could have run the actual publication as well, which
would have fail in the same way, so this makes no difference. However,
if the dry run succeeds, we recompile each package one more time from
scratch, performing the exact same work for publication, which is
wasteful. At this point of the release script, we clearly intend to
release and we should just use `cargo publish` normally.
Now that Topiary is published and crates.io (and tree-sitter-nickel as
well), we move away from git revision and use proper public versions for
those. Doing so, we can also crap the special casing that was done
during release to disable those dependencies and we'll now be able to
release on crates.io a version with formatting enabled.
Co-authored-by: Yann Hamdaoui <yann.hamdaoui@tweag.io>
* Make base branch configurable in release script
Instead of hardcoding master as the base branch for a new release, put
it inside a shell variable than can be overriden (at least by modifying
the script) for patch releases, where it's more common to start off a
specific branch which isn't master (but the previous release + selected
bug fixes).
Additionally, add a missing phase before releasing to crates.io: get rid
of `lsp-harness` dependencies in the LSP.
* [release.sh] update to 1.4.1
* Update scripts/release.sh
* Only update relevant deps in Cargo.lock
The release script needs to update Cargo.lock after having updated
Cargo.toml with the new version of various Nickel crates.
Indeed, we are running some checks via Nix, and Nix builds are sandboxed
without network access. This requires the Nix build to use `cargo
--frozen --offline`, which in turns require the Cargo.lock file to be
up-to-date or it will fail.
The previous version of the release script would do a simple `cargo
update`, which has the default of updating all dependencies. This commit
changes that to only udpate dependencies which we might have bumped.
* Add pause to commit release notes in release script
* Fix release script for LSP
The release script was missing a part: the removal of `lsp-harness` (a
local, unpublished util crates) from the nickel-lang-lsp package priori
to publication to crates.io.
Passing by, other small improvements and fixes are included as well.
* [release.sh] update to 1.4.0
* Specify 'do not commit' when updating release notes
* release guide: add missing step for lsp-harness removal
* Add a release script
First draft of a release script to automate most of the painful details
of releasing a new version of Nickel and associated crates.
* Update the releasing guide with new script
* Minor fixes to release script
* Change casing, use namerefs and other small improvements
* Make sure to checkout master before release process
* Make release.sh executable
* Shellcheck pass
* Various fixes and improvement
* More polishing of the release script
* Try sed-based Cargo.toml modifications
* WIP
* Fix cleanup actions after using it add
* Improve confirmation printing
* Fix undefined variable and cleanup actions
* Fix cleanup of ./Cargo.toml
* Fix jq invocation and circular reference warning
* Run cargo update during release script to fix Nix flake check
* Add cargo as requirement for release.sh
* Comment out effectful commands, fix Cargo.lock cleanup
* Add topiary cleaning to the release script
* [TODROP] disable nix flake check for rapid iteration
* [TODROP] Disable cleanup for debugging purpose
* Tentative fix of the remove Topiary dep step
* Avoid trailing commas when removing Topiary
* Improve output of removing topiary dependencies
* Fix issues with format removal step
* More fixing of the format removal step
* Put back actual git commands and cleanup
* Move release script to its own directory
* Make release script re-entrant once release branch exists
* Cd to the root git dir in the release script
* Disable some shellcheck warnings, remove trailing spaces, add missing git switch
* Run checks even if the release branch exists
* Describe Topiary removal in the releasing guide
* Update RELEASING.md
Co-authored-by: jneem <joeneeman@gmail.com>
* Cosmetic improvements to release script
---------
Co-authored-by: jneem <joeneeman@gmail.com>
