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.

Sign up for GitHub

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

Jump to bottom

Check for "Test Plan Approved" Label in PRs #13718

Merged
merged 3 commits into from
May 11, 2022
Merged

Check for "Test Plan Approved" Label in PRs #13718

merged 3 commits into from
May 11, 2022

Conversation

hetunandu
Copy link
Member

Check for the label "Test Plan Approved" in PRs

If this does not exist the PR check will fail

@vercel
Copy link

vercel bot commented May 10, 2022 •
edited
Loading

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

Name Status Preview Updated
appsmith ✅ Ready (Inspect) Visit Preview May 10, 2022 at 11:25AM (UTC)

@github-actions
Copy link

Unable to find test scripts. Please add necessary tests to the PR.

@hetunandu hetunandu added the Test Plan Approved Manual/Cypress tests covers changes made on the PR. Else, add skip-testPlan label if not applicable label May 10, 2022
@hetunandu hetunandu added BE Coders Pod Issues related to users writing code to fetch and update data and removed BE Coders Pod Issues related to users writing code to fetch and update data Test Plan Approved Manual/Cypress tests covers changes made on the PR. Else, add skip-testPlan label if not applicable labels May 10, 2022
@hetunandu hetunandu added BE Coders Pod Issues related to users writing code to fetch and update data Test Plan Approved Manual/Cypress tests covers changes made on the PR. Else, add skip-testPlan label if not applicable labels May 10, 2022
@hetunandu hetunandu added FE Coders Pod Issues related to users writing javascript in appsmith Test Plan Approved Manual/Cypress tests covers changes made on the PR. Else, add skip-testPlan label if not applicable and removed BE Coders Pod Issues related to users writing code to fetch and update data Test Plan Approved Manual/Cypress tests covers changes made on the PR. Else, add skip-testPlan label if not applicable labels May 11, 2022
@hetunandu hetunandu merged commit 97c5aee into release May 11, 2022
@hetunandu hetunandu deleted the Check-for-"Test-Plan-Approved"-Label-in-PRs branch May 11, 2022 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nidhi-nair nidhi-nair nidhi-nair approved these changes

Assignees

@ajinkyakulkarni ajinkyakulkarni

@nidhi-nair nidhi-nair

Labels
FE Coders Pod Issues related to users writing javascript in appsmith Test Plan Approved Manual/Cypress tests covers changes made on the PR. Else, add skip-testPlan label if not applicable
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@hetunandu @nidhi-nair @ajinkyakulkarni