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 CI conditions again #64

Merged
merged 8 commits into from
Aug 2, 2023
Merged

Fix CI conditions again #64

merged 8 commits into from
Aug 2, 2023

Conversation

WarningImHack3r
Copy link
Member

@WarningImHack3r WarningImHack3r commented Aug 1, 2023

Understood why deploy and perf-check are skipped thanks to my StackOverflow question and tested the solution on a test repository, finally works fine.

Also (in this PR so we are DONE with config PRs):

  • Add pnpx git-removed-branches -p to the Husky post-checkout to automatically delete local branches that have been merged remotely on checkout
  • Downgrade @inlang/sdk-js even more to hopefully fix random vite build failures (I guess it didn't happen in 0.6.x)
    • I couldn't reproduce the bug locally or even with act so I cross fingers
  • Remove the dedicated ci:lint task because Prettier seems to behave correctly now

@WarningImHack3r WarningImHack3r marked this pull request as ready for review August 2, 2023 19:22
@WarningImHack3r WarningImHack3r merged commit 20cda93 into main Aug 2, 2023
5 checks passed
@WarningImHack3r WarningImHack3r deleted the fix-ci-conditions-2 branch August 2, 2023 19:56
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