WindowsDevicePortalWrapper
0.9.0.0
A client library that wraps the Windows Device Portal REST APIs.
|
The Windows Device Portal Wrapper welcomes contributions from the community.
For things like fixing typos and small bug fixes, you can skip this step.
If your change is more than a simple fix, please don't just create a big pull request. Instead, start by opening an issue describing the problem you want to solve and how you plan to approach the problem. This will let us have a brief discussion about the problem and, hopefully, identify some potential pitfalls before too much time is spent.
Note: If you wish to work on something that already exists on our backlog, you can use that work item as your proposal.
The Windows Device Portal Wrapper uses the StyleCop code analysis tool to ensure code consistency and readability. This step is required for the WindowsDevicePortalWrapper folder and is optional (though highly recommended) for test applications.
If there is a StyleCop issue that you believe does not need to be enforced, please add the suppression entry either to your code or the Settings.StyleCop file in the appropriate folder. This will highlight the rule change and allow the community to comment.
The commits in your pull request should tell a story about how the code got from point A to point B. Good stories are edited, so you'll want to rebase your commits so that they tell a good story.
Each commit should build and pass all of the tests. If you want to add new tests for functionality that's not yet written, ensure the tests are added disabled.
Don't forget to run git diff –check to catch those annoying whitespace changes.
Please follow the established Git convention for commit messages. The first line is a summary in the imperative, about 50 characters or less, and should not end with a period. An optional, longer description must be preceded by an empty line and should be wrapped at around 72 characters. This helps with various outputs from Git or other tools.
You can update message of local commits you haven't pushed yet using git commit –amend or git rebase –interactivewith reword command.
Start a GitHub pull request to merge your topic branch into the main repository's master branch. (If you are a Microsoft employee and are not a member of the Microsoft organization on GitHub yet, please contact the DDE team via e-mail for instructions before starting your pull request. There's some process stuff you'll need to do ahead of time.) If you haven't contributed to a Microsoft project before, you may be asked to sign a contribution license agreement. A comment in the PR will let you know if you do.
The project maintainers will review your changes. We aim to review all changes within three business days. Address any review comments, force push to your topic branch, and post a comment letting us know that there's new stuff to review.
If the pull request review goes well, a project maintainer will merge your changes. Thank you for helping improve the Windows Device Portal Wrapper!
For maintainers, since it's a total hack
The Windows Device Portal Wrapper uses Doxygen to automatically generate code documentation directly from the source code. Any changes to existing or new classes or methods should also update the documentation.
Alternate: