admin/Kae7in-perfview
1
0
Fork 0
mirror of https://github.com/Kae7in/perfview synced 2026-08-09 12:09:44 +00:00
Code Issues Packages Projects Releases Wiki Activity

Update CONTRIBUTING.md

Browse Source
This commit is contained in:
Vance Morrison
2016-05-06 09:24:27 -07:00
parent 2c0ed9bfac
commit e7d1c0bde7
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
CONTRIBUTING.md
+3 -1
View File
@@ -52,7 +52,9 @@ In general all features need pre-vetting. The rule here is simple. Don't do a
you
1. Are willing to throw it away (e.g. it was more effort to get it vetted) OR
2. You have vetted it with us by creating an issue.
2. You have vetted it with us by creating an issue, and got positive confirmation from the
discussion that you are on the right track. If it is a big change, you should ongoing
discussion on the issue to insure that you stay on track.
Performance improvements are often a point of contention. Improvements that make the code
smaller/simpler are great, but often this is not the case. If you are adding complexity as