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: upload /formatters test coverage #9753

Merged
merged 1 commit into from
Aug 8, 2023

Conversation

almeidx
Copy link
Member

@almeidx almeidx commented Aug 6, 2023

Please describe the changes this PR makes and why it should be merged:
Upload the test coverage of the formatters package to codecov

Status and versioning classification:

@almeidx almeidx requested a review from a team as a code owner August 6, 2023 13:33
@vercel
Copy link

vercel bot commented Aug 6, 2023

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 Aug 6, 2023 1:33pm
discord-js-guide ⬜️ Ignored (Inspect) Visit Preview Aug 6, 2023 1:33pm

@github-actions
Copy link

github-actions bot commented Aug 6, 2023

⚡️ Lighthouse report for the changes in this PR:

Category Score
🟠 Performance 88
🟢 Accessibility 97
🟢 Best practices 100
🟠 SEO 75
🔴 PWA 30

Lighthouse ran on https://discord-js-git-fork-almeidx-formatters-coverage-discordjs.vercel.app/

@almeidx almeidx changed the title chore: upload /formatters test coverage ci: upload /formatters test coverage Aug 6, 2023
@kodiakhq kodiakhq bot merged commit cf72d42 into discordjs:main Aug 8, 2023
19 checks passed
@almeidx almeidx deleted the formatters-coverage branch August 8, 2023 07:33
@Jiralite Jiralite removed the tests label Aug 8, 2023
Tynopia added a commit to cyberpurge-net/discord.js that referenced this pull request Aug 8, 2023
ci: upload /formatters test coverage (discordjs#9753)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

5 participants