Skip to content

Commit 2099030

Browse files
delastevehansl
authored andcommitted
docs: update documentation for schematics and contributing
1 parent f45b89f commit 2099030

3 files changed

Lines changed: 71 additions & 24 deletions

File tree

.github/ISSUE_TEMPLATE.md

Lines changed: 43 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,43 @@
1+
<!--
2+
IF YOU DON'T FILL OUT THE FOLLOWING INFORMATION YOUR ISSUE MIGHT BE CLOSED WITHOUT INVESTIGATING
3+
-->
4+
### Bug Report or Feature Request (mark with an `x`)
5+
```
6+
- [ ] bug report -> please search issues before submitting
7+
- [ ] feature request
8+
```
9+
10+
### Area
11+
```
12+
- [ ] devkit
13+
- [ ] schematics
14+
```
15+
16+
### Versions
17+
<!--
18+
Output from: `node --version` and `npm --version`.
19+
Windows (7/8/10). Linux (incl. distribution). macOS (El Capitan? Sierra? High Sierra?)
20+
-->
21+
22+
23+
### Repro steps
24+
<!--
25+
Simple steps to reproduce this bug.
26+
Please include: commands run, packages added, related code changes.
27+
A link to a sample repo would help too.
28+
-->
29+
30+
31+
### The log given by the failure
32+
<!-- Normally this include a stack trace and some more information. -->
33+
34+
35+
### Desired functionality
36+
<!--
37+
What would like to see implemented?
38+
What is the usecase?
39+
-->
40+
41+
42+
### Mention any other details that might be useful
43+
<!-- Please include a link to the repo if this is related to an OSS project. -->

CONTRIBUTING.md

Lines changed: 19 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# Contributing to Angular DevKit
22

3-
We would love for you to contribute to Angular CLI and help make it even better
3+
We would love for you to contribute to DevKit and help make it even better
44
than it is today! As a contributor, here are the guidelines we would like you
55
to follow:
66

@@ -18,7 +18,7 @@ Help us keep Angular open and inclusive. Please read and follow our [Code of Con
1818

1919
## <a name="question"></a> Got a Question or Problem?
2020

21-
Please, do not open issues for the general support questions as we want to keep GitHub issues for bug reports and feature requests. You've got much better chances of getting your question answered on [StackOverflow](https://stackoverflow.com/questions/tagged/angular-cli) where the questions should be tagged with tag `angular-cli`.
21+
Please, do not open issues for the general support questions as we want to keep GitHub issues for bug reports and feature requests. You've got much better chances of getting your question answered on [StackOverflow](https://stackoverflow.com/questions/tagged/angular-devkit) where the questions should be tagged with tag `angular-devkit`.
2222

2323
StackOverflow is a much better place to ask questions since:
2424

@@ -38,8 +38,7 @@ If you find a bug in the source code or a mistake in the documentation, you can
3838
## <a name="feature"></a> Want a Feature?
3939
You can *request* a new feature by [submitting an issue](#submit-issue) to our [GitHub
4040
Repository][github]. If you would like to *implement* a new feature, please submit an issue with
41-
a proposal for your work first, to be sure that we can use it. Angular CLI is in developer preview
42-
and we are not ready to accept major contributions ahead of the full release.
41+
a proposal for your work first, to be sure that we can use it.
4342
Please consider what kind of change it is:
4443

4544
* For a **Major Feature**, first open an issue and outline your proposal so that it can be
@@ -53,9 +52,9 @@ and help you to craft the change so that it is successfully accepted into the pr
5352

5453
Before you submit an issue, please search the issue tracker, maybe an issue for your problem already exists and the discussion might inform you of workarounds readily available.
5554

56-
We want to fix all the issues as soon as possible, but before fixing a bug we need to reproduce and confirm it. In order to reproduce bugs we will systematically ask you to provide a minimal reproduction scenario using `ng new repro-app`. Having a reproducible scenario gives us wealth of important information without going back & forth to you with additional questions like:
55+
We want to fix all the issues as soon as possible, but before fixing a bug we need to reproduce and confirm it. Having a reproducible scenario gives us wealth of important information without going back & forth to you with additional questions like:
5756

58-
- version of Angular CLI used
57+
- version of Angular DevKit used
5958
- `.angular-cli.json` configuration
6059
- 3rd-party libraries and their versions
6160
- and most importantly - a use-case that fails
@@ -66,13 +65,13 @@ We will be insisting on a minimal reproduce scenario in order to save maintainer
6665

6766
Unfortunately we are not able to investigate / fix bugs without a minimal reproduction, so if we don't hear back from you we are going to close an issue that don't have enough info to be reproduced.
6867

69-
You can file new issues by filling out our [new issue form](https://github.com/angular/angular-cli/issues/new).
68+
You can file new issues by filling out our [new issue form](https://github.com/angular/devkit/issues/new).
7069

7170

7271
### <a name="submit-pr"></a> Submitting a Pull Request (PR)
7372
Before you submit your Pull Request (PR) consider the following guidelines:
7473

75-
* Search [GitHub](https://github.com/angular/angular-cli/pulls) for an open or closed PR
74+
* Search [GitHub](https://github.com/angular/devkit/pulls) for an open or closed PR
7675
that relates to your submission. You don't want to duplicate effort.
7776
* Please sign our [Contributor License Agreement (CLA)](#cla) before sending PRs.
7877
We cannot accept code without this.
@@ -84,7 +83,7 @@ Before you submit your Pull Request (PR) consider the following guidelines:
8483

8584
* Create your patch, **including appropriate test cases**.
8685
* Follow our [Coding Rules](#rules).
87-
* Run the full Angular CLI test suite, as described in the [developer documentation][dev-doc],
86+
* Run the full Angular DevKit test suite, as described in the [developer documentation][dev-doc],
8887
and ensure that all tests pass (coming soon).
8988
* Commit your changes using a descriptive commit message that follows our
9089
[commit message conventions](#commit). Adherence to these conventions
@@ -101,10 +100,10 @@ Before you submit your Pull Request (PR) consider the following guidelines:
101100
git push origin my-fix-branch
102101
```
103102

104-
* In GitHub, send a pull request to `angular-cli:master`.
103+
* In GitHub, send a pull request to `devkit:master`.
105104
* If we suggest changes then:
106105
* Make the required updates.
107-
* Re-run the Angular CLI test suites to ensure tests are still passing.
106+
* Re-run the Angular DevKit test suites to ensure tests are still passing.
108107
* Rebase your branch and force push to your GitHub repository (this will update your Pull Request):
109108

110109
```shell
@@ -155,7 +154,7 @@ To ensure consistency throughout the source code, keep these rules in mind as yo
155154

156155
We have very precise rules over how our git commit messages can be formatted. This leads to **more
157156
readable messages** that are easy to follow when looking through the **project history**. But also,
158-
we use the git commit messages to **generate the Angular CLI change log**.
157+
we use the git commit messages to **generate the Angular DevKit change log**.
159158

160159
### Commit Message Format
161160
Each commit message consists of a **header**, a **body** and a **footer**. The header has a special
@@ -187,19 +186,19 @@ Must be one of the following:
187186
* **fix**: A bug fix
188187
* **perf**: A code change that improves performance
189188
* **refactor**: A code change that neither fixes a bug nor adds a feature
190-
* **style**: Changes that do not affect the meaning of the code (white-space, formatting, missing
191-
semi-colons, etc)
189+
* **style**: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
192190
* **test**: Adding missing tests or correcting existing tests
193191

194192
### Scope
195193
The scope should be the name of the npm package affected as perceived by the person reading changelog generated from the commit messages.
196194

197195
The following is the list of supported scopes:
198196

199-
* **@angular/cli**
200-
* **@ngtools/json-schema**
201-
* **@ngtools/logger**
202-
* **@ngtools/webpack**
197+
* **@angular-devkit/core**
198+
* **@angular-devkit/build-optimizer**
199+
* **@angular-devkit/schematics**
200+
* **@schematics/angular**
201+
* **@schematics/schematics**
203202

204203
There are currently a few exceptions to the "use package name" rule:
205204

@@ -240,8 +239,8 @@ changes to be accepted, the CLA must be signed. It's a quick process, we promise
240239
[commit-message-format]: https://docs.google.com/document/d/1QrDFcIiPjSLDn3EL15IJygNPiHORgU1_OOAqWjiDU5Y/edit#
241240
[corporate-cla]: http://code.google.com/legal/corporate-cla-v1.0.html
242241
[dev-doc]: https://github.com/angular/angular-cli#development-hints-for-working-on-angular-cli
243-
[GitHub]: https://github.com/angular/angular-cli
242+
[GitHub]: https://github.com/angular/devkit
244243
[gitter]: https://gitter.im/angular/angular-cli
245244
[individual-cla]: http://code.google.com/legal/individual-cla-v1.0.html
246245
[js-style-guide]: https://google.github.io/styleguide/jsguide.html
247-
[stackoverflow]: http://stackoverflow.com/questions/tagged/angular-cli
246+
[stackoverflow]: http://stackoverflow.com/questions/tagged/angular-devkit

packages/angular_devkit/schematics/collection-schema.json

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,21 +6,26 @@
66
"properties": {
77
"schematics": {
88
"type": "object",
9+
"description": "A map of schematic names to schematic details",
910
"properties": {
1011
"additionalProperty": {
1112
"type": "object",
1213
"properties": {
1314
"factory": {
14-
"type": "string"
15+
"type": "string",
16+
"description": "A folder or file path to the schematic factory"
1517
},
1618
"description": {
17-
"type": "string"
19+
"type": "string",
20+
"description": "A description for the schematic"
1821
},
1922
"extends": {
20-
"type": "string"
23+
"type": "string",
24+
"description": "An schematic override. It can be a local schematic or from another collection (in the format 'collection:schematic')"
2125
},
2226
"schema": {
23-
"type": "string"
27+
"type": "string",
28+
"description": "Location of the schema.json file of the schematic"
2429
}
2530
},
2631
"required": [

0 commit comments

Comments
 (0)