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

Update PR template to encourage adding unit tests #12205

Merged
merged 1 commit into from
Aug 5, 2024

Conversation

malinajirka
Copy link
Contributor

This PR encourages developers to add unit tests or provide a brief explanation when they believe it's not worth it.

@malinajirka malinajirka added the category: tooling Anything that involves building & maintaining the project, including scripts, `Fastfile`, etc. label Aug 2, 2024
@malinajirka malinajirka added this to the 19.9 milestone Aug 2, 2024
@malinajirka malinajirka requested a review from AnirudhBhat August 2, 2024 08:07
@wpmobilebot
Copy link
Collaborator

📲 You can test the changes from this Pull Request in WooCommerce-Wear Android by scanning the QR code below to install the corresponding build.
App Name WooCommerce-Wear Android
Platform⌚️ Wear OS
FlavorJalapeno
Build TypeDebug
Commitb811d5d
Direct Downloadwoocommerce-wear-prototype-build-pr12205-b811d5d.apk

@wpmobilebot
Copy link
Collaborator

📲 You can test the changes from this Pull Request in WooCommerce Android by scanning the QR code below to install the corresponding build.

App Name WooCommerce Android
Platform📱 Mobile
FlavorJalapeno
Build TypeDebug
Commitb811d5d
Direct Downloadwoocommerce-prototype-build-pr12205-b811d5d.apk

@AnirudhBhat AnirudhBhat self-assigned this Aug 5, 2024
Copy link
Contributor

@AnirudhBhat AnirudhBhat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for adding this @malinajirka!

@AnirudhBhat AnirudhBhat merged commit d2be662 into trunk Aug 5, 2024
15 checks passed
@AnirudhBhat AnirudhBhat deleted the update-PR-template branch August 5, 2024 05:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: tooling Anything that involves building & maintaining the project, including scripts, `Fastfile`, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants