Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .github/copilot-instructions.md
Original file line number Diff line number Diff line change
Expand Up @@ -108,6 +108,11 @@ go tool cover -html=coverage.out
- **Go formatting**: Code MUST pass `gofmt`
- **Proto changes**: Update generated code when modifying `.proto` files
- **Security**: gNMI server handles authentication — be security-conscious
- **PR description template**: Fill out all sections of the [PR template](.github/pull_request_template.md) when submitting a pull request:
- **Description of PR**: Summary of the change, motivation/context, reviewer entry point, and dependencies; reference issues with `fixes #xxxx` / `closes #xxxx`.
- **Type of change**: Mark the box(es) that apply — bug fix, new feature, refactor / cleanup, documentation update, test improvement.
- **Approach**: Motivation; how you did it; how you verified/tested it; any platform-specific notes.
- **Documentation**: Link to wiki / doc updates relevant to new features or test cases.

## Common Patterns

Expand Down
95 changes: 28 additions & 67 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
@@ -1,84 +1,45 @@
<!--
Please make sure you've read and understood our contributing guidelines:
https://github.com/Azure/SONiC/blob/gh-pages/CONTRIBUTING.md
Please make sure you've read and understood our contributing guidelines;
https://github.com/Azure/SONiC/blob/gh-pages/CONTRIBUTING.md

** Make sure all your commits include a signature generated with `git commit -s` **

If this is a bug fix, make sure your description includes "fixes #xxxx", or
"closes #xxxx" or "resolves #xxxx"

Please provide the following information:
Please provide following information to help code review process a bit easier:
-->

#### Why I did it

#### How I did it

#### How to verify it

#### Which release branch to backport (provide reason below if selected)

### Description of PR
<!--
- Note we only backport fixes to a release branch, *not* features!
- Please also provide a reason for the backporting below.
- If you request a backport/cherry-pick, provide both:
1. A GitHub issue or Microsoft ADO work item describing the failure.
2. Test evidence from the target release branch(es) requested below,
not only from master.
- e.g.
- [x] 202006
- Please include a summary of the change and which issue is fixed.
- Please also include relevant motivation and context. Where should reviewer start? background context?
- List any dependencies that are required for this change.
-->

- [ ] 202305
- [ ] 202311
- [ ] 202405
- [ ] 202411
- [ ] 202505
- [ ] 202511
- [ ] 202605

Tracking issue/work item for backport/cherry-pick request (GitHub issue or Microsoft ADO):
Failure type: <!-- day-one issue / regression / other -->
Summary:
Fixes # (issue)

#### Tested branch
### Type of change

<!--
- Select each branch where the change was tested.
- If you request a backport/cherry-pick, select the base branch and the tested
target release branch(es).
- Fill x for your type of change.
- e.g.
- [x] Bug fix
-->

- [ ] master
- [ ] 202305
- [ ] 202311
- [ ] 202405
- [ ] 202411
- [ ] 202505
- [ ] 202511
- [ ] 202605
- [ ] N/A
- [ ] Bug fix
- [ ] New feature
- [ ] Refactor / cleanup
- [ ] Documentation update
- [ ] Test improvement

#### Test result
### Approach
#### What is the motivation for this PR?

<!--
- Provide the tested image version and test evidence for each selected branch.
- e.g.
- master: 20260716.01 - <test result or link>
- 202605: 20260531.42 - <test result or link>
-->
#### How did you do it?

#### Description for the changelog
<!--
Write a short (one line) summary that describes the changes in this
pull request for inclusion in the changelog:
-->
#### How did you verify/test it?

#### Any platform specific information?

#### Link to config_db schema for YANG module changes
### Documentation
<!--
Provide a link to config_db schema for the table for which YANG model
is defined
Link should point to correct section on https://github.com/Azure/SONiC/wiki/Configuration.
(If it's a new feature, new test case)
Did you update documentation/Wiki relevant to your implementation?
Link to the wiki page?
-->

#### A picture of a cute animal (not mandatory but encouraged)

Loading