-
Notifications
You must be signed in to change notification settings - Fork 48
Contribution Guidelines
This can be seen as a draft and should probably be improved more in the future
Create a new discussion at GitHub
Feel free to post an issue here on GitHub. Please have a look at the good bug filing template for asp.net found at https://github.com/aspnet/Home/wiki/Functional-bug-template
Download the code, compile and have a look around. There are several sample projects in Samples repository to play around with
What to contribute with:
- If you are starting you can look for issues marked as Up For Grabs or which looks simple.
- If you find an issue which you want to provide a fix for then please do add a comment that you want to or will investigate it.
- The documentation is currently very sparse and lacking, please feel free to contribute. You can always post issues with proposed documentation.
- Not all tests currently run and some are non-trivial to get going, so any help in getting more tests passing or improving the getting started experience are welcome.
Try to follow the existing coding style. Make sure that you can compile the project with your changes using release build without any new compilation warning.
We generally try to follow Microsoft .NET Framework Design Guidelines
All new public, protected and internal methods should be documented, using standard "{{///}}" comments. Make sure that you can compile the project with your changes using release build without any new compilation warning.
- Make sure that the first line always contains a short descriptive summary of the changes.
- The remainting rows can be used to provide more details about the changes.
- You can use "Fix # BUG TITLE" if bug title is describing and the contribution is a "small" one-commit pull request
- If the commits
Example of a commit message for a fix which only containst a single commit:
Fix #X THE BUG DESCRIPTION /summary of changes
Commit detail 1
Commit detail 2
Commit detail 3
For multi part commits, it is a good idea to embed the issue number in the commit message, but it is not mandatory at the moment.
[Issue #232](Issue-#232) Short summary of changes
Commit detail 1
Commit detail 2
Commit detail 3
Please take your time and add test for any new functionality added.