Skip to content

build because all is no more#264

Merged
ericsciple merged 1 commit intoactions:masterfrom
gnawhleinad:npm-run-build
May 31, 2020
Merged

build because all is no more#264
ericsciple merged 1 commit intoactions:masterfrom
gnawhleinad:npm-run-build

Conversation

@gnawhleinad
Copy link
Copy Markdown
Contributor

fixes #263.

echo Troubleshooting
echo ----------------------------------------
echo "::error::Unstaged changes detected. Locally try running: git clean -ffdx && npm ci && npm run all"
echo "::error::Unstaged changes detected. Locally try running: git clean -ffdx && npm ci && npm run build"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

needs npm format now too:

    echo "::error::Unstaged changes detected. Locally try running: git clean -ffdx && npm ci && npm run format && npm run build"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

incrementally better. so i'll go ahead and merge :)

thanks!

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

also i pushed an update to include npm run format

@ericsciple ericsciple merged commit 65865e1 into actions:master May 31, 2020
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.

npm run all is missing

2 participants