What tests are expected when submitting a pull request? #173671
Replies: 7 comments
This comment was marked as off-topic.
This comment was marked as off-topic.
-
|
🕒 Discussion Activity Reminder 🕒 This Discussion has been labeled as dormant by an automated system for having no activity in the last 60 days. Please consider one the following actions: 1️⃣ Close as Out of Date: If the topic is no longer relevant, close the Discussion as 2️⃣ Provide More Information: Share additional details or context — or let the community know if you've found a solution on your own. 3️⃣ Mark a Reply as Answer: If your question has been answered by a reply, mark the most helpful reply as the solution. Note: This dormant notification will only apply to Discussions with the Thank you for helping bring this Discussion to a resolution! 💬 |
Beta Was this translation helpful? Give feedback.
-
|
Before submitting a pull request, you should run the following tests to ensure your changes don’t break anything:
Running these tests before submitting a pull request helps maintain the integrity of the codebase and speeds up the review process. |
Beta Was this translation helpful? Give feedback.
-
|
🕒 Discussion Activity Reminder 🕒 This Discussion has been labeled as dormant by an automated system for having no activity in the last 60 days. Please consider one the following actions: 1️⃣ Close as Out of Date: If the topic is no longer relevant, close the Discussion as 2️⃣ Provide More Information: Share additional details or context — or let the community know if you've found a solution on your own. 3️⃣ Mark a Reply as Answer: If your question has been answered by a reply, mark the most helpful reply as the solution. Note: This dormant notification will only apply to Discussions with the Thank you for helping bring this Discussion to a resolution! 💬 |
Beta Was this translation helpful? Give feedback.
-
|
Before submitting a pull request, you should run the full unit test suite and add tests for any new or changed logic. Integration or end-to-end tests should be run if the change affects multiple components or user flows. It’s also important to run linters, type checks, and ensure the project builds successfully. For bug fixes, add a regression test to prevent the issue from reoccurring, and do a quick manual sanity check where applicable. |
Beta Was this translation helpful? Give feedback.
-
|
Before you hit “Create Pull Request,” it’s a good idea to double check a few things to make sure your changes are solid:
|
Beta Was this translation helpful? Give feedback.
-
|
Before opening a PR, I usually do a quick sanity check:
|
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
Select Topic Area
Question
Body
Before you send your code changes for review (a pull request), what kinds of tests should you run to make sure your changes don’t break anything?
Beta Was this translation helpful? Give feedback.
All reactions