-
Notifications
You must be signed in to change notification settings - Fork 106
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
docs(version): ✨ Pulling version number from helia. #290
Conversation
@@ -33,8 +33,9 @@ | |||
"release": "run-s build docs:no-publish npm:release docs", | |||
"npm:release": "aegir exec --bail false npm -- publish", | |||
"release:rc": "aegir release-rc", | |||
"docs": "NODE_OPTIONS=--max_old_space_size=8192 aegir docs -- --exclude packages/interop --excludeExternals", | |||
"docs:no-publish": "NODE_OPTIONS=--max_old_space_size=8192 aegir docs --publish false -- --exclude packages/interop" | |||
"update:version": "node scripts/update-version.js", |
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.
this does not commit anything back to main, only the docs.
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.
Please can we be careful about the commit messages used as they will affect final version numbers of shipping software.
This is not a feature so feat:
is inappropriate. Instead, please choose from the available tags - docs:
seems like the obvious one to me.
If necessary please review the Conventional Commits documentation.
This is a strange fix because v2.0.3 is the current version of helia which is compliant with v2.0.0 of the helia interface. In theory we'll see lots of patch releases of helia but very few releases to the interface. The API docs are monorepo docs rather than specific to a given module and the version number comes from the root package.json. Perhaps there's a config key for the root typedoc.json to remove display of the monorepo version as it doesn't make much sense to the consumer? Then we don't need contortions like the one introduced in this PR. |
c777d83
to
a43823a
Compare
@achingbrain However, I still think there is value in having the version number that corresponds to the latest helia. The interface correctly describes the current version as well. |
It was being hard-coded in aegir's |
Alternative: #296 |
Closing in favour of #296 |
Fixes the package version on the docs hosted on helia.io.
helia
.package.json
This should fix the version in the next release.