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

Automate documentation creation #2860

Closed
ilyavolodin opened this issue Jun 27, 2015 · 15 comments
Closed

Automate documentation creation #2860

ilyavolodin opened this issue Jun 27, 2015 · 15 comments
Labels
accepted There is consensus among the team that this change meets the criteria for inclusion archived due to age This issue has been archived; please open a new issue for any further discussion core Relates to ESLint's core APIs and features enhancement This change enhances an existing feature of ESLint

Comments

@ilyavolodin
Copy link
Member

I would like to streamline documentation create a bit. Now that we have schemas, I would like to create a built task that would export schemas and append available options at the end of each rule documentation.I also think we can take it even further, in order to help people search google for errors they get as a result of linting, I think we should include all possible error messages into exportable object that would also be appended to rule documentation. We also can remove "README.md" all together, and generated it from exported object. I will create a WIP pull request with proposed changes for a few rules.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@ilyavolodin ilyavolodin added triage An ESLint team member will look at this issue soon needs bikeshedding Minor details about this change need to be discussed labels Jun 27, 2015
@lo1tuma
Copy link
Member

lo1tuma commented Jun 27, 2015

Some error messages are created dynamically, so I don't think it is possible to automatically document every possible error message.

@ilyavolodin
Copy link
Member Author

Most of the error messages are created by substitution, that's why I used {0} in my example. If the messages is completely generated (I think there might be one or two rules that do that), then oh well... It's better to have most of the messages then none.

@nzakas
Copy link
Member

nzakas commented Jun 27, 2015

I think this might end up being a bit too pie-in-the-sky, but feel free to explore. :)

@IanVS
Copy link
Member

IanVS commented Jul 20, 2015

append available options at the end of each rule documentation.

It seems to me like option documentation is important enough that it should be fairly early in the rule docs. Imagine needing to scroll to the very bottom of object-curly-spacing to find the options.

@ilyavolodin
Copy link
Member Author

Agree. But without knowing how the document is structured, it's hard to insert auto-created pieces in the middle.

@IanVS
Copy link
Member

IanVS commented Jul 20, 2015

Could we manually put a placeholder in all the rule docs? Perhaps stub out the header "### Rule Options" or similar?

@mysticatea
Copy link
Member

Or I think to generate TOC automatically is good for large rules.

@IanVS
Copy link
Member

IanVS commented Jul 20, 2015

I agree, looks like it was discussed in #1073, but I don't see it live on the site.

@ilyavolodin
Copy link
Member Author

It is live. Or at least anchors on all headers are. If you mouse over any header, you should see a link icon pop up.

@IanVS
Copy link
Member

IanVS commented Jul 20, 2015

You're right, I wasn't clear. The anchors are there, but there are no TOC.

@nzakas
Copy link
Member

nzakas commented Jul 21, 2015

I don't think there are enough sections on rule docs to have a TOC... Most have three, four sections at most. I'm not a big fan of boilerplate like that.

@IanVS
Copy link
Member

IanVS commented Jul 21, 2015

This is getting off topic, but how about some of the larger pages like Configuring ESLint?

@nzakas
Copy link
Member

nzakas commented Jul 21, 2015

I think the larger pages should be broken up into smaller ones

@ilyavolodin ilyavolodin added enhancement This change enhances an existing feature of ESLint core Relates to ESLint's core APIs and features and removed triage An ESLint team member will look at this issue soon labels Sep 16, 2015
@alberto alberto added the evaluating The team will evaluate this issue to decide whether it meets the criteria for inclusion label May 15, 2016
@nzakas nzakas added accepted There is consensus among the team that this change meets the criteria for inclusion and removed evaluating The team will evaluate this issue to decide whether it meets the criteria for inclusion needs bikeshedding Minor details about this change need to be discussed labels Aug 1, 2016
@kaicataldo
Copy link
Member

Is this still being worked on?

@ilyavolodin
Copy link
Member Author

Second part is done (Index.md). The rest of it is controversial enough to hold off for now.

@eslint-deprecated eslint-deprecated bot locked and limited conversation to collaborators Feb 7, 2018
@eslint-deprecated eslint-deprecated bot added the archived due to age This issue has been archived; please open a new issue for any further discussion label Feb 7, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
accepted There is consensus among the team that this change meets the criteria for inclusion archived due to age This issue has been archived; please open a new issue for any further discussion core Relates to ESLint's core APIs and features enhancement This change enhances an existing feature of ESLint
Projects
None yet
Development

No branches or pull requests

7 participants