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

fix: fix steps in CI test #99

Merged
merged 1 commit into from
Nov 25, 2024
Merged

fix: fix steps in CI test #99

merged 1 commit into from
Nov 25, 2024

Conversation

massongit
Copy link
Contributor

editor/package.json

Lines 31 to 32 in 00dba75

"build": "lerna run build",
"test": "lerna run build && lerna run test",

In CI test, yarn build runs twice because yarn test includes yarn build.
Therefore, I change steps in CI test to yarn install -> yarn test -> npm run dist chrome.

@azu azu added the Type: CI Changes to CI configuration files and scripts label Nov 25, 2024
@azu azu merged commit 1ec7b3f into textlint:master Nov 25, 2024
7 checks passed
@massongit massongit deleted the patch-1 branch November 25, 2024 23:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: CI Changes to CI configuration files and scripts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants