Remove obsolete tests references from CONTRIBUTING.md - #136
Merged
Conversation
Member
|
如有需要，请更新相关文档并为你的修改添加测试。 这句话也要删掉 @copilot |
Co-authored-by: hzhangxyz <11623447+hzhangxyz@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Remove reference to tests folder from contributing MD
Remove obsolete tests folder reference from CONTRIBUTING.md
Dec 18, 2025
hzhangxyz
requested changes
Dec 18, 2025
Member
There was a problem hiding this comment.
"If necessary, please update documentations and add tests for your changes." 这句话也删掉吧.
Co-authored-by: hzhangxyz <11623447+hzhangxyz@users.noreply.github.com>
Contributor
Author
Copilot
AI
changed the title
Remove obsolete tests folder reference from CONTRIBUTING.md
Remove obsolete tests references from CONTRIBUTING.md
Dec 18, 2025
hzhangxyz
approved these changes
Dec 18, 2025
hzhangxyz
marked this pull request as ready for review
December 18, 2025 21:00
Copilot stopped work on behalf of
hzhangxyz due to an error
December 18, 2025 21:01
There was a problem hiding this comment.
Pull request overview
This PR removes obsolete references to the tests folder from CONTRIBUTING.md, as the tests folder has been removed from the repository.
- Removed mention of
testsfolder from the Code Structure section - Removed "and add tests" from the contribution guidelines section
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| The comments in the code are expected to be enough for other developers to understand your code. | ||
| Please add docstrings to the code in NumPy style. | ||
| If necessary, please update documentations and add tests for your changes. | ||
| If necessary, please update documentations for your changes. |
There was a problem hiding this comment.
The word "documentations" should be "documentation" (singular form). In English, "documentation" is typically used as an uncountable noun.
Suggested change
| If necessary, please update documentations for your changes. | |
| If necessary, please update documentation for your changes. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The tests folder has been removed from the repository, but CONTRIBUTING.md still referenced it in multiple places.
Changes
tests" from the Code Structure section (line 15)qmp." instead of mentioning bothqmpandtestsfoldersOriginal prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.