-
Notifications
You must be signed in to change notification settings - Fork 80
Create and Submit PullRequest
The first step before starting to create a Pull Request (PR) is to ensure you've opened a Discussions or are Submitting an Issue describing the changes you wish to make. This allows the community and Toolkit team to be aware of your proposal and guide your new changes to success! If your change is only minor and extremely focused (like fixing a typo), you may skip this step.
If this is your first time submitting a PR on GitHub, following GitHub's official guide on Creating a pull request.
Anyone can create a Pull Request by forking the Windows Community Toolkit repository. Here is how you can Create a Pull Request from fork
Once you fork the Windows Community Toolkit repo, it is essential to create all changes in a feature branch ( for example add-listview-extensions
) of your forked repository. If you have the changes in the forked feature branch you can create a Pull Request in the main Windows Community Toolkit where your changes will be merged to the main branch.
Once the PR is created and submitted, we will be able to see the associated PR issue and will assist and provide feedback if necessary.
If you are not certain about the Pull Request submitted; our team along with the community members are here to review the work and highlight any changes the PR might require to fulfill any required obligation.
For every contribution, you must:
- Test your code with the supported SDKs
- Follow the quality guidance, general rules, and naming convention
- Target main branch (or an appropriate release branch if appropriate for a bug fix)
- Follow the Windows Community Toolkit PR Template
- If adding a new feature
- Before starting coding, you should open an issue and start discussing with the community to see if your idea/feature is interesting enough.
- Add Documentation for the feature, see Documentation
- Add or update a sample for the Sample app
- If creating a new sample, create a new icon by following the Thumbnail Style Guide and templates
- Add or update unit tests and integration tests, see Testing
A PR must be validated by at least two core members before being merged, though having other community members validate your PR is always welcome and encouraged, see Review PR.
Please make certain that the PR successfully passes all three status check requirements including Toolkit-CI, WIP, license/cla. If it fails and you are aware of the reasoning please fix the error. If you are unaware of the reasoning please escalate by commenting in the PR, and someone from the team will investigate the build error.
If the PR passes all the requirements above and reviewers have signed off, the PR can be merged.
Once merged, you can get a pre-release package of the toolkit by adding this (Azure DevOps feed | Gallery) to your Visual Studio.
- Home π
- Welcome π
- Principles βοΈ
- Roadmap πΊ
- .NET Foundation
- Why Microsoft supports this project
- License π
- Getting Started π
- Sample App π±
- NuGet Packages π¦
- Preview Packages π
- Toolkit Labs π§ͺ
- Questions β
- Discussions π₯
- Submitting an Issue βοΈ
- Good First Issue π
- Help Wanted π
- Bug Fixes π
- Feature Requests π«
- Create and Submit PullRequest π
- Documentation π
- Review PR π
- Avoid Roadblocks π§
- Required Dependencies π
- Quick Start β‘
- Sample Development π±
- Testing π§ͺ
- Accessibility Guideline βΏ
- Coding Style and Conventions β
- Building XAML Controls π
- CI and Infrastructure πΎ
- Updating ToolkitLabs.dev π
- Migration Guide from v7 to v8 π
- How the Project is Organized ποΈ
- Join the Toolkit Organization πͺ
- Hall of Fame π