Skip to content

Check for existing critcmp before cargo install - #105

Open
juntyr wants to merge 1 commit into
boa-dev:mainfrom
juntyr:check-cargo-install
Open

Check for existing critcmp before cargo install#105
juntyr wants to merge 1 commit into
boa-dev:mainfrom
juntyr:check-cargo-install

Conversation

@juntyr

@juntyr juntyr commented Aug 21, 2023

Copy link
Copy Markdown

Fixes #104

@juntyr

juntyr commented Aug 21, 2023

Copy link
Copy Markdown
Author

Running npx prettier --check . locally runs successfully, since I have prettier v2.8.8 installed, as mentioned in package.json. However, it seems like the CI action does not use the pinned version but installs the latest one, v3.0.2, which finds style issues unrelated to my change.

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.

cargo install critcmp installs if install is cached by CI runner

1 participant