-
Notifications
You must be signed in to change notification settings - Fork 323
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Reorganize developer documentation (#2576)
- Loading branch information
Showing
34 changed files
with
183 additions
and
223 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
File renamed without changes.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
File renamed without changes.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,62 +1,7 @@ | ||
# Dev Documentation | ||
|
||
## Fork, Clone, Branch and Create your PR | ||
|
||
Once you've discussed your proposed feature/fix/etc. with a team member, and you've agreed on an approach or a spec has been written and approved, it's time to start development: | ||
|
||
1. Fork the repo if you haven't already | ||
1. Clone your fork locally | ||
1. Create & push a feature branch <!--1. Create a [Draft Pull Request (PR)](https://github.blog/2019-02-14-introducing-draft-pull-requests/)--> | ||
1. Work on your changes | ||
|
||
## Rules | ||
|
||
- **Follow the pattern of what you already see in the code.** | ||
- [Coding style](style.md). | ||
- Try to package new ideas/components into libraries that have nicely defined interfaces. | ||
- Package new ideas into classes or refactor existing ideas into a class as you extend. | ||
- When adding new classes/methods/changing existing code: add new unit tests or update the existing tests. | ||
<!-- | ||
## GitHub Workflow | ||
- Before starting to work on a fix/feature, make sure there is an open issue to track the work. | ||
- Add the `In progress` label to the issue, if not already present also add a `Cost-Small/Medium/Large` estimate and make sure all appropriate labels are set. | ||
- If you are a community contributor, you will not be able to add labels to the issue, in that case just add a comment saying that you started to work on the issue and try to give an estimate for the delivery date. | ||
- If the work item has a medium/large cost, using the markdown task list, list each sub item and update the list with a check mark after completing each sub item. | ||
- When opening a PR, follow the PR template. | ||
- When you'd like the team to take a look, (even if the work is not yet fully-complete), mark the PR as 'Ready For Review' so that the team can review your work and provide comments, suggestions, and request changes. It may take several cycles, but the end result will be solid, testable, conformant code that is safe for us to merge. | ||
- When the PR is approved, let the owner of the PR merge it. For community contributions the reviewer that approved the PR can also merge it. | ||
- Use the `Squash and merge` option to merge a PR, if you don't want to squash it because there are logically different commits, use `Rebase and merge`. | ||
- We don't close issues automatically when referenced in a PR, so after the PR is merged: | ||
- mark the issue(s), that the PR solved, with the `Resolution-Fix-Committed` label, remove the `In progress` label and if the issue is assigned to a project, move the item to the `Done` status. | ||
- don't close the issue if it's a bug in the current released version since users tend to not search for closed issues, we will close the resolved issues when a new version is released. | ||
- if it's not a code fix that affects the end user, the issue can be closed (for example a fix in the build or a code refactoring and so on). | ||
--> | ||
## Compiling Dev Home | ||
|
||
### Compiling Source Code | ||
|
||
There are two ways to compile locally. | ||
|
||
- Open the Developer Command Prompt for Visual Studio | ||
- Run `Build` from Dev Home's root directory. You can pass in a list of platforms/configurations | ||
- The Dev Home MSIX will be in your repo under `AppxPackages\x64\debug` | ||
|
||
Alternatively | ||
|
||
- Open `DevHome.sln` in Visual Studio, in the `Solutions Configuration` drop-down menu select `Release` or `Debug`, from the `Build` menu choose `Build Solution`. | ||
|
||
### Dev Home framework | ||
|
||
The Dev Home project contains the wrapping framework for Dev Home. | ||
It's responsible for: | ||
- Loading the individual Dev Home tools. | ||
- Loading out-of-process extensions. | ||
- Loading the main window and creating a skeleton UI. | ||
|
||
|
||
## More | ||
|
||
Before you dive into developing for Dev Home, check out these resources: | ||
- [Dev Home Architecture](./architecture.md) | ||
- [Dev Home Tools](./tools.md) | ||
- [Dev Home Extensions](./extensions.md) | ||
- [Dev Home Tools](./tools/readme.md) | ||
- [Dev Home Extensions](./extensions/extensions.md) | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
4 changes: 2 additions & 2 deletions
4
docs/sampleConfigurations/Templates/Introduction/C#/README.md
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
4 changes: 2 additions & 2 deletions
4
docs/sampleConfigurations/Templates/Introduction/C++/README.md
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.