Skip to content

Api Codegen TypeScript: Clean cache before formating#2091

Open
kevin-legion wants to merge 1 commit intomainfrom
fix-prettier-api-codegen
Open

Api Codegen TypeScript: Clean cache before formating#2091
kevin-legion wants to merge 1 commit intomainfrom
fix-prettier-api-codegen

Conversation

@kevin-legion
Copy link

Hopefully fixes this CI error https://github.com/legion-labs/legion/runs/6987927767?check_suite_focus=true that occurs randomly.

The error seems to be on the NPM cli side: npm/cli#4096.

The fix itself is rather extreme as we basically clean the NPM cache each time the prettier command fails.

@kevin-legion kevin-legion force-pushed the fix-prettier-api-codegen branch from 785ae26 to c894ef6 Compare June 21, 2022 20:15
@kevin-legion kevin-legion marked this pull request as ready for review June 21, 2022 20:16
@kevin-legion
Copy link
Author

@tltran-legion I wonder, is the CI machine cache preserved between builds? That could explain the error 😕

@kevin-legion kevin-legion force-pushed the fix-prettier-api-codegen branch from c894ef6 to 9fb0a5e Compare June 21, 2022 20:20
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.

2 participants