Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

chore(dev): shorten test plan in PR template#62962

Merged
jhchabran merged 1 commit into
mainfrom
jh/shorten_pr_template.md
May 29, 2024
Merged

chore(dev): shorten test plan in PR template#62962
jhchabran merged 1 commit into
mainfrom
jh/shorten_pr_template.md

Conversation

@jhchabran

@jhchabran jhchabran commented May 29, 2024

Copy link
Copy Markdown
Contributor

Fixes https://github.com/sourcegraph/devx-support/issues/990

Changelog

  • Test plan has been in place for a long time, so we can get rid of the detailed instructions.

Test plan

Irrelevant, just a PR template update. CI.

@cla-bot cla-bot Bot added the cla-signed label May 29, 2024
@jhchabran jhchabran force-pushed the jh/shorten_pr_template.md branch from 295007f to 76a06e7 Compare May 29, 2024 13:09
@jhchabran jhchabran marked this pull request as ready for review May 29, 2024 13:11
@jhchabran jhchabran requested review from a team and varungandhi-src May 29, 2024 13:11

## Test plan

<!-- All pull requests REQUIRE a test plan: https://docs-legacy.sourcegraph.com/dev/background-information/testing_principles -->

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

A shame this is in docs-legacy 😅

@varungandhi-src varungandhi-src left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks!

@jhchabran jhchabran merged commit 2ecc169 into main May 29, 2024
@jhchabran jhchabran deleted the jh/shorten_pr_template.md branch May 29, 2024 14:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants