Skip to content

Github action to publish the package #3

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

Merged
merged 1 commit into from
Mar 22, 2023
Merged

Conversation

jstastny
Copy link

@jstastny jstastny commented Mar 22, 2023

This adds new github action "publish", which is triggered any time tag starting with v (for example v1.3.5) is pushed.

The action then builds and publishes version according to the tag name. So for example for tag v1.3.5, this action will publish 1.3.5. The workflow expects the tag to follow semantic versioning -- internally it uses yarn version (from yarn classic -- because it is used by the upstream) and it fails on invalid / non-semantic versions.

Given this is triggered by tag, the workflow does not push back the altered package.json with the bumped version.

The package is pushed to GitHub package registry in this repository.

@jstastny jstastny force-pushed the js/publish-github-package branch 19 times, most recently from db9a2bc to 07e6e53 Compare March 22, 2023 11:14
@jstastny jstastny changed the title WIP: Github action Github action to publish the package Mar 22, 2023
@jstastny jstastny requested a review from vire March 22, 2023 11:16
- run: yarn install
- run: yarn npm:prepublish
- name: Set the package name
run: npm pkg set --workspace packages/server name=@deepnote/sql-language-server
Copy link

Choose a reason for hiding this comment

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

suggestion: I would maybe add a line that this does the renaming of original package.json name field :) it's not that super obvious but maybe not important as well...

Copy link
Author

Choose a reason for hiding this comment

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

Sure. I can add comments (and was planning to). I assume by "line" you mean "comment", correct?

Copy link

@vire vire left a comment

Choose a reason for hiding this comment

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

good job!

@jstastny jstastny force-pushed the js/publish-github-package branch from 07e6e53 to 690c386 Compare March 22, 2023 15:23
@jstastny jstastny force-pushed the js/publish-github-package branch from 3aba010 to 48ce253 Compare March 22, 2023 20:21
@jstastny jstastny merged commit d6af39b into release Mar 22, 2023
@the21st the21st deleted the js/publish-github-package branch March 23, 2023 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants