Skip to content
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

CI: Automate release workflow #102

Merged
merged 6 commits into from
Nov 19, 2023
Merged

CI: Automate release workflow #102

merged 6 commits into from
Nov 19, 2023

Conversation

Finii
Copy link
Collaborator

@Finii Finii commented Nov 18, 2023

[why]
Releasing is too hard.

[how]
The maintainer needs to set a new version in the package.json. Afterwards the release draft creator can be manually triggered on the Actions tab of the Github repo.

This will draft a release with a new font generated from the current vectors. And the current commit will be git tagged with the release version.

Fixes: #29

[why]
Releasing is too hard.

[how]
The maintainer needs to set a new version in the package.json.
Afterwards the release draft creator can be manually triggered on
the Actions tab of the Github repo.

This will draft a release with a new font generated from the current
vectors. And the current commit will be git tagged with the release
version.

Signed-off-by: Fini Jastrow <ulf.fini.jastrow@desy.de>
@Finii
Copy link
Collaborator Author

Finii commented Nov 18, 2023

I imitated the contents of the release v1.0.1 files. Not sure that all that files are really useful. Otoh do not know if this npm thing need it.

@Finii
Copy link
Collaborator Author

Finii commented Nov 18, 2023

Just noticed that the files are not automagically compressed.
I'm always confused, sometimes they are, sometimes not.
I think the normal action artifacts are always packed, and the release artifacts not... 🤔 😬

image

Signed-off-by: Fini Jastrow <ulf.fini.jastrow@desy.de>
Signed-off-by: Fini Jastrow <ulf.fini.jastrow@desy.de>
Signed-off-by: Fini Jastrow <ulf.fini.jastrow@desy.de>
Signed-off-by: Fini Jastrow <ulf.fini.jastrow@desy.de>
Signed-off-by: Fini Jastrow <ulf.fini.jastrow@desy.de>
@Finii Finii marked this pull request as ready for review November 19, 2023 14:59
@Finii
Copy link
Collaborator Author

Finii commented Nov 19, 2023

Ok, this seems ready. I tested everything except the final publish-to-npm.

Will try to use the workflow to get v1.0.1 published, this will involve some tricks, though.

@Finii Finii merged commit 21ab871 into master Nov 19, 2023
@Finii
Copy link
Collaborator Author

Finii commented Nov 19, 2023

Test on v1.0.1, success:

image

image

@Finii
Copy link
Collaborator Author

Finii commented Nov 19, 2023

@Finii
Copy link
Collaborator Author

Finii commented Nov 19, 2023

Actual assets missing?? Working on it...

image

@Finii
Copy link
Collaborator Author

Finii commented Nov 19, 2023

Fixed via the following commits. Needed to increase version because NPM does not allow any other change

  • 881cc5e Rerelase fixed NPM package, no change
  • 8d6d767 CI: Fix missing assets in npm

image

Sorry 😒

@Finii Finii deleted the feature/release-workflow branch November 19, 2023 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Auto-publish to npm
2 participants