-
Notifications
You must be signed in to change notification settings - Fork 3.9k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
chore: require docstring on public members (#4645)
* chore: require docstring on public members Many people (still) contribute PRs with undocumented members, or undocumented classes, or undocumented defaults, because there is no tool support to check they're not forgetting. We used to have `@default` checking on props types for L2s, but not for interfaces that are used INSIDE L2 props structs, or that are used as props types for non-constructs. Add two new checks: * Requirement to document top-level types and public members. * Requirement to supply `@default` for optional properties on structs. Which supersede the previous one. Obviously, we have many currently undocumented APIs which are added to the ignore list. Fixes #2286. * Update region-info awslint script * Update with new undocumented APIs * More undefaulted props * Ignore some more undocumented public APIs that got merged in the mean time
- Loading branch information
1 parent
3459982
commit dc8061a
Showing
65 changed files
with
2,203 additions
and
167 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
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
Oops, something went wrong.