-
Notifications
You must be signed in to change notification settings - Fork 223
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
publish
option does not support multiple bash commands
#225
Comments
I have the same problem, the action run build cmd but skips publish cmd. I workaround it by merging the build script and the publish to an npm script. lawvs/poi-plugin-quest-2@caf000f Incorrect action see https://github.com/lawvs/poi-plugin-quest-2/actions/runs/3262382860/workflow - name: Create Release Pull Request or Publish to npm
id: changesets
uses: changesets/action@v1
with:
publish: npm run build && npm publish # <- the publish script was skipped
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
NPM_TOKEN: ${{ secrets.NPM_TOKEN }}
And the correct output is like in here https://github.com/lawvs/poi-plugin-quest-2/actions/runs/3262903759/jobs/5360750330 |
kwaa
added a commit
to importantimport/shiraha
that referenced
this issue
Mar 27, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Example yaml
Expected behavior
The action runs
echo "hello world"
and thenyarn changeset publish
, unless the first command exits with non-zero. Thus, syntax parsing matchespackage.json
→scripts
.Observed behavior
The action runs
echo "hello world" "&&" "yarn" "changeset" "publish"
.Context
Action version: v1.3.0
Related source:
action/src/run.ts
Lines 76 to 89 in 8c3f5f5
The text was updated successfully, but these errors were encountered: