Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Documentation doesn't mention inherited members #196

Closed
RomainMuller opened this issue Aug 24, 2018 · 3 comments
Closed

Documentation doesn't mention inherited members #196

RomainMuller opened this issue Aug 24, 2018 · 3 comments
Assignees
Labels
feature-request A feature should be added or improved.

Comments

@RomainMuller
Copy link
Contributor

The documentation generated by jsii-pacmak does not include references to inherited members that were not overridden. This makes the documentation appear a little confusing and makes discovery of a full API surface more complicated.

For example, the s3.Bucket class inherits most of it's interesting members from s3.BucketRef, but this is not entirely obvious (particularly to a newcomer). The documentation should include inherited members as part of a class' documentation in one way or another.

@eladb
Copy link
Contributor

eladb commented Aug 28, 2018

I agree, we should include the full docs for inherited members!

@Doug-AWS
Copy link

Does that mean we'll also show all Construct members for all that extend Construct, such as BucketRef?

@eladb
Copy link
Contributor

eladb commented Aug 28, 2018

Yes, ideally indicating where these methods came from...

@RomainMuller RomainMuller self-assigned this Sep 18, 2018
RomainMuller added a commit that referenced this issue Sep 18, 2018
Add a documentation line outlining the parent declaration of overridden
members as well as inherited members that are not overridden locally.
This should make browsing the documentation a lot nicer.

Inherited and overridden statements in the documentation always refer to
the fully qualified name of the super statement.

Fixes #196
RomainMuller added a commit that referenced this issue Sep 19, 2018
Add a documentation line outlining the parent declaration of overridden
members as well as inherited members that are not overridden locally.
This should make browsing the documentation a lot nicer.

Inherited and overridden statements in the documentation always refer to
the fully qualified name of the super statement.

Fixes #196
eladb pushed a commit that referenced this issue Sep 20, 2018
Bug Fixes
======================
 * Sphinx generated incorrect type references for display ([#232](#232)) ([b664805](b664805))
* **jsii:** Defaulted parameters were not rendered as optional ([#234](#234)) ([578bf9c](578bf9c)), closes [#233](#233)
* **jsii:** Don't skip emit on TS errors when in "watch" mode ([#236](#236)) ([30d1491](30d1491)), closes [#235](#235)
* **jsii:** Optional `any` represented as required ([#237](#237)) ([91074f3](91074f3)), closes [#230](#230)

Features
======================
 * **sphinx:** allow readme file to define sphinx header and reorganize topic ([#229](#229)) ([405da9c](405da9c)), closes [#228](#228) [#185](#185)
* Document overriden/inherited members ([#238](#238)) ([7a6278a](7a6278a)), closes [#196](#196)
@srchase srchase added feature-request A feature should be added or improved. and removed enhancement labels Jan 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request A feature should be added or improved.
Projects
None yet
Development

No branches or pull requests

4 participants