Skip to content

Conversation

@AhyoungRyu
Copy link
Contributor

What is this PR for?

Recently I added auto generated Table of Contents by #1031 and searching feature by #1266. So I wrote some guide to docs/CONTRIBUTING.md about how to use and how they can be applied to docs page. Hope this helps to new Zeppelin (docs) contributors :)

What type of PR is it?

Documentation

Todos

  • - Update docs/README.md

What is the Jira issue?

ZEPPELIN-1323

How should this be tested?

Just click View button on the top of the file diff :)

Screenshots (if appropriate)

Questions:

  • Does the licenses files need update? no
  • Is there breaking changes for older versions? no
  • Does this needs documentation? no

@Leemoonsoo
Copy link
Member

CI failure looks unrelated to this change.

Looks great to me!

If you're going to create new pages, there are some spots you need to add the location of the page.

- **Dropdown menu in navbar**: add your docs location to [_navigation.html](https://github.com/apache/zeppelin/blob/master/docs/_includes/themes/zeppelin/_navigation.html)
- **Main index**: add your docs below [What is the next?](http://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/#what-is-the-next) section in [index.md](http://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/#what-is-the-next) with short description. No need to do this if the page is for **Interpreters**.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@felixcheung Yeah right. Thx for pointing it out!

@felixcheung
Copy link
Member

LGTM

@AhyoungRyu
Copy link
Contributor Author

Is this good to go now? :)

@felixcheung
Copy link
Member

felixcheung commented Aug 18, 2016 via email

@asfgit asfgit closed this in ff2465b Aug 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants