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

ci: optimize tests #10231

Merged
merged 1 commit into from Apr 27, 2024
Merged

ci: optimize tests #10231

merged 1 commit into from Apr 27, 2024

Conversation

Syjalo
Copy link
Contributor

@Syjalo Syjalo commented Apr 26, 2024

Please describe the changes this PR makes and why it should be merged:
This PR makes tests check run first than linter, because tests check is faster and fails quite often

Status and versioning classification:

  • Code changes have been tested against the Discord API, or there are no code changes
  • I know how to update typings and have done so, or typings don't need updating
  • This PR only includes non-code changes, like changes to documentation, README, etc.

@Syjalo Syjalo requested a review from iCrawl as a code owner April 26, 2024 22:24
Copy link

vercel bot commented Apr 26, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Ignored Deployments
Name Status Preview Comments Updated (UTC)
discord-js ⬜️ Ignored (Inspect) Visit Preview Apr 26, 2024 10:24pm
discord-js-guide ⬜️ Ignored (Inspect) Visit Preview Apr 26, 2024 10:24pm

@Jiralite Jiralite changed the title chore: optimize tests ci: optimize tests Apr 26, 2024
@Jiralite Jiralite added the ci label Apr 26, 2024
@iCrawl iCrawl merged commit dae897b into discordjs:main Apr 27, 2024
6 checks passed
@Syjalo Syjalo deleted the chore/optimize-tests branch April 27, 2024 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants