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

feat(workspace): update PR template #900

Merged
merged 2 commits into from
Dec 6, 2024
Merged

Conversation

asharonbaltazar
Copy link
Collaborator

Purpose

I'm tired of the old PR template, so I've decided to renew it

Copy link
Contributor

github-actions bot commented Dec 5, 2024

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 71.09% 5719 / 8044
🔵 Statements 70.52% 6049 / 8577
🔵 Functions 65.05% 1720 / 2644
🔵 Branches 40.02% 1017 / 2541
File CoverageNo changed files found.
Generated in workflow #825 for commit 0d647a5 by the Vitest Coverage Report Action

@benjaminpaige
Copy link
Collaborator

I like it,especially the screenshot part, i always enjoy a good screenshot in a pr. also even just some 'polite' reminders would be good. are all the tests passing? if you removed any code did you remove anything related to that code we no longer need? i dont kow something like that. even maybe..... "did you run this new code through a llm to see if there is anyway it could be improved? Are all types complete and correct?

i know i am more guilty than most of these atrocities.... but that still doesn't mean it wouldn't be good to include them. lol

anyways, i like it. keep it simple, but relevent. 👍

Copy link
Contributor

🎉 This PR is included in version 1.5.0-val.98 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants