Skip to content

Commit 88d5ef3

Browse files
authored
Updated CONTRIBUTING.md to account for new project planning board (#10104)
Update CONTRIBUTING.md
1 parent e014c37 commit 88d5ef3

File tree

1 file changed

+4
-11
lines changed

1 file changed

+4
-11
lines changed

.github/CONTRIBUTING.md

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -10,11 +10,7 @@ all community interactions must abide by the [Code of Conduct](https://github.co
1010

1111
## Finding issues you can help with
1212
Looking for something to work on?
13-
Issues marked [``triage approved``](https://github.com/files-community/Files/issues?q=is%3Aopen+is%3Aissue+label%3A%22triage+approved%22)
14-
are a good place to start.
15-
16-
You can also check the [``help wanted``](https://github.com/files-community/Files/issues?q=is%3Aopen+is%3Aissue+label%3A%22help+wanted%22) tag to find
17-
other issues to help with. If you're interested in working on a fix, leave a comment to let everyone know and to help
13+
Issues marked as ready to build are a good place to start. If you're interested in working on an issue, leave a comment to let everyone know and to help
1814
avoid duplicated effort from others.
1915

2016
## Contributions we accept
@@ -29,11 +25,9 @@ improvements which address the top issues reported by Files users. Some general
2925
pull request.
3026
* **DO** write a good description for your pull request. More detail is better. Describe *why* the change is being
3127
made and *why* you have chosen a particular solution. Describe any manual testing you performed to validate your change.
32-
* **DO NOT** submit a PR unless it is linked to an Issue marked
33-
[`triage approved`](https://github.com/files-community/Files/issues?q=is%3Aopen+is%3Aissue+label%3A%22triage+approved%22).
34-
This enables us to have a discussion on the idea before anyone invests time in an implementation.
28+
* **DO NOT** submit a PR unless it the connected issue is marked as ready to build or approved indirectly by an org member.
29+
This enables us to have a discussion on the idea before anyone invests time on the implementation.
3530
* **DO NOT** merge multiple changes into one PR unless they have the same root cause.
36-
* **DO NOT** submit pure formatting/typo changes to code that has not been modified otherwise.
3731

3832
> Submitting a pull request for an approved Issue is not a guarantee it will be approved.
3933
> The change must meet our high bar for code quality, architecture, and performance.
@@ -53,8 +47,7 @@ After installing WinAppDriver, you will need to run the Files.Package at least o
5347
Once you have the debug version registered, you can run the tests from the test explorer in Visual Studio.
5448

5549
## Review Process
56-
After submitting a pull request, members of the Files team will review your code. We will
57-
assign the request to an appropriate reviewer. Any member of the community may
50+
After submitting a pull request, members of the Files team will review your code. Any member of the community may
5851
participate in the review, but at least one member of the Files team will ultimately approve
5952
the request.
6053

0 commit comments

Comments
 (0)