Semantic Release is not updating the `master` branch version

[ashblue]

Releases are not updating the root package.json version. Investigate why. Open a ticket with Oyster or the package assembler if the issue is in the corresponding package.

[favoyang]

I guess the reason could be you're running the npm plugin against the generated dist/ folder. The original package.json is not get updated. So you need to tweak the ordering, by updating package.json at the right place first, then run the build script. You also need to commit the changed package.json via Git plugin.

Here's a working .releaserc:
https://github.com/favoyang/unity-addressable-importer/blob/master/.releaserc.json (json format, but simliar)

[favoyang]

Please try install semantic-release and run it with --dryrun to verify your modification https://semantic-release.gitbook.io/semantic-release/usage/configuration#dryrun

[ashblue]

Well it's working in some of my other repos so I'll have to do a little digging. Yeah, it could quite possibly be the ordering in the release file.

[ashblue]

Finally had some time to dig into this. The issue is directly tied to Oyster Package Manager. It seems a version change on the Git semver plugin requires specifying the package.json to check it in. Need to do some more digging there.

https://github.com/ashblue/oyster-package-generator

[favoyang]

You need to specify the assets parameter of @semantic-release/git (in .releaserc.json) if the package is not located at the root path.

https://github.com/semantic-release/git

And if the CHANGELOG is not located at the root path, you need to specify the changelogFile path of @semantic-release/changelog as well.

https://github.com/semantic-release/changelog

[ashblue]

I found the issue and it has to do with the NPM plugin running after the Git plugin for Semantic Release. I'm also having to rewrite the build compiler to hard write version values to master before deploy. Going to take me some time to get the CLI tool stable to do this via Oyster (WIP).

[favoyang]

Nice catch.

[ashblue]

This issue should now be resolved as of #59

[favoyang]

This issue should now be resolved as of #59

@ashblue thanks for the update. So everything should be settled down once this been merged into the master branch?

[favoyang]

And am I right that you will not change the package name of these package.json files to make it different?

• https://github.com/ashblue/fluid-behavior-tree/blob/develop/package.json com.fluid.behavior-tree
• https://github.com/ashblue/fluid-behavior-tree/blob/develop/Assets/com.fluid.behavior-tree/package.json com.fluid.behavior-tree

[ashblue]

The underlying package manager Oyster relies on the root package.json for a lot of operations. As the package manager itself is written in Node.js (same package tech Unity opted into). Multiple package.json files aren't unusual in Node.js projects. It could be renamed or removed into a separate branch, but it would be a lot of work to do so. If there was some sort of underlying config file to specify the pacakge.json I could easily fix this. It would have to be that or a pull request on Oyster to create a separate branch at the right time in the template files. Not against the idea at all, just don't have time to write the feature atm (pandemic was not kind to me the past year).