Skip to content

Conversation

achingbrain
Copy link
Member

@achingbrain achingbrain commented May 12, 2025

Turns out ignoring .gitignore while adding files to the release commit is a feature not a bug so specify the list of files to add.

We didn't trigger it in the past because we didn't add the lockfile to the build cache so it wasn't copied into the container for the release build step. Now we do in order to support projects (like apps) that want a lockfile so we need to specify the asset files in the config.

Refs: semantic-release/git#56

We can probably revert/update if semantic-release/git#527 ever gets merged.

Turns out ignoring `.gitignore` while adding files to the release commit is a feature not a bug so specify the list of files to add.

Refs: semantic-release/git#56

We can probably revert if semantic-release/git#527 ever gets merged.
Copy link
Member

@rvagg rvagg left a comment

Choose a reason for hiding this comment

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

👍 thanks for working this out; I'll have to copy this to dag-pb, dag-json, js-car, etc. which were all hit with this

@rvagg rvagg merged commit 2013bda into master May 20, 2025
19 checks passed
@rvagg rvagg deleted the chore/omit-lockfile-from-release-commit branch May 20, 2025 02:53
rvagg added a commit to multiformats/js-multiformats that referenced this pull request May 20, 2025
rvagg added a commit to multiformats/js-multiformats that referenced this pull request May 20, 2025
rvagg added a commit to multiformats/js-multiformats that referenced this pull request May 20, 2025
rvagg added a commit to multiformats/js-multiformats that referenced this pull request May 20, 2025
rvagg added a commit to multiformats/js-multiformats that referenced this pull request May 21, 2025
rvagg added a commit to ipld/js-dag-pb that referenced this pull request May 21, 2025
github-actions bot pushed a commit that referenced this pull request May 22, 2025
## [9.2.4](v9.2.3...v9.2.4) (2025-05-22)

### Bug Fixes

* address lint errors ([467d98c](467d98c))

### Trivial Changes

* omit lockfile from release commit ([#126](#126)) ([2013bda](2013bda))
* remove package-lock ([6445f53](6445f53))

### Dependencies

* **dev:** bump aegir from 46.0.5 to 47.0.10 ([c0b2879](c0b2879))
Copy link

🎉 This PR is included in version 9.2.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants