It seems that people do not find the exp-run flag when opening a PR.
Details
Diff Detail
- Repository
- rD FreeBSD doc repository - subversion
- Lint
No Lint Coverage - Unit
No Test Coverage - Build Status
Buildable 4812 Build 4869: Jenkins Build Doc Build 4868: arc lint + arc unit
Event Timeline
en_US.ISO8859-1/articles/committers-guide/article.xml | ||
---|---|---|
4877 | s/aswers/answers/ |
en_US.ISO8859-1/articles/committers-guide/article.xml | ||
---|---|---|
4828 | "What is the procedure" is still a bit redundant. Also, "test" is helpful to make the purpose clearer. How can an experimental test build of the ports tree (<emphasis>exp-run</emphasis>) be requested? | |
4834 | &a.portmgr; prefers that an exp-run is performed before patches with a large impact to ports or the base system are committed. | |
4838 | Please, not "you". <para>Full package builds will be done with the patches provided by the submitter, and the submitter is required to fix detected problems before commit.</para> | |
4853 | No, please, no "you". Fill in the bug report as normal. | |
4854 | Negative logic: if !(forget). Use positive logic: Remember to attach the patch. | |
4861 | This is more confusing because it asks for the opposite of what is needed. Also, "you are good" is ...not good. Please remind me about this later when I have more energy. | |
4865 | This assumes a certain page layout which might not always be that way. Can probably just say "Set the flags to..." | |
4868 | No, please not "your" unless absolutely necessary. Hovering the mouse over any field shows more details. | |
4877 | s/the &a.portmgr;/&a.portmgr/ (I think). s/answers back/replies/ | |
4878 | "fallout" might not be clearly defined to the reader. | |
4879 | Please, no "you" or "your" except when needed. The fallout might be a bug in the original patch or other ports that need to be fixed. Both need to be addressed. Update the patch to repair all the fallout and repeat. |
en_US.ISO8859-1/articles/committers-guide/article.xml | ||
---|---|---|
4828 | It sounds better, most other questions start with "What ..." so I went with it :-) | |
4834 | Is this the polite way of saying "you have to" ? | |
4838 | There are "you" and "your" all over the place here, it always was less formal than the rest of this article. | |
4877 | Without "the" it reads: When Ports Management Team <portmgr@FreeBSD.org> replies,... |
en_US.ISO8859-1/articles/committers-guide/article.xml | ||
---|---|---|
4834 | The polite way is "must", like An exp-run must be completed before patches with a significant ports impact are committed. | |
4838 | Yes, but we should be working on that. It's not to make it more formal, it's to make the sentences more direct. | |
4877 | Oops, sorry, thought it was a bare email address. Yes, it needs the "the". |