-
Notifications
You must be signed in to change notification settings - Fork 259
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
Create GitHub release and git tag with custom publishing #269
Comments
This is basically what I was also doing. I think that nowadays you should be able to make private packages taggable and that should automatically resolve the issue - I agree that it's not exactly obvious but perhaps it's just a documentation issue. This requirement is vaguely mentioned here |
I see! So having a "dummy" publish script that satisfies the
|
I'm not 100% sure but I think that with this
I'm definitely open for docs PRs 😉 |
After some investigation and testing we found that including |
Have you veen able to look into the PR @Andarist? I'll be more than happy to make changes or discuss the implementation. |
Problem
We are using
changesets/action@v1
for versioning and publishing of a non-npm application (one "root" application, no packages). In our workflow we have implemented custom publishing as referred to (https://github.com/changesets/action#custom-publishing). This works as expected, apart from GitHub releases and git tags not being created. In a (separate) monorepo publishing to NPM, this is working as expected.According to https://github.com/changesets/changesets/blob/main/docs/versioning-apps.md, it seems like this should be possible.
We have tested that it works with
changeset/cli
.As far as I can tell, after some investigation,
createRelease
is not called ifpublish:
is not specified in the workflow.Current workaround
After looking into the
runPublish
method, we found that puttingpublish: yarn dummy-publish
in the workflow and"dummy-release": "echo \"New tag:" && exit 0"
under scripts inpackage.json
. This satisfies the condition.action/src/run.ts
Lines 133 to 138 in 595655c
The text was updated successfully, but these errors were encountered: