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

Add 'contributing' and 'code reference' docs to repo #29

Merged
merged 21 commits into from
Aug 10, 2020

Conversation

kakirastern
Copy link

@kakirastern kakirastern commented Aug 4, 2020

Description

Add the remaining main documentation to docs, including contributing and code reference, to supplement the user's guide and developer's guide already added to repo.

============

Code of Conduct
---------------
Copy link
Collaborator

Choose a reason for hiding this comment

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

I would probably just link to the sunpy one.

Copy link
Author

Choose a reason for hiding this comment

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

Okay, sure

Copy link
Author

Choose a reason for hiding this comment

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

I ended up doing something a little different as it felt a bit odd to use the same licence for the Code of Conduct for glue-solar with sunpy, so I adapted the sunpy one. Hope this arrangement is okay with you.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Should be ok.

docs/code-reference.rst Outdated Show resolved Hide resolved
docs/code-reference.rst Outdated Show resolved Hide resolved
tox.ini Outdated Show resolved Hide resolved
docs/conf.py Outdated Show resolved Hide resolved
tox.ini Outdated Show resolved Hide resolved
tox.ini Outdated Show resolved Hide resolved
@kakirastern kakirastern changed the title [WIP] Add other docs Add 'contributing' and 'code reference' docs to repo Aug 10, 2020
Comment on lines +69 to +93
# -- Turn on nitpicky mode for sphinx (to warn about references not found) ----
#
# nitpicky = True
# nitpick_ignore = []
#
# Some warnings are impossible to suppress, and you can list specific references
# that should be ignored in a nitpick-exceptions file which should be inside
# the docs/ directory. The format of the file should be:
#
# <type> <class>
#
# for example:
#
# py:class astropy.io.votable.tree.Element
# py:class astropy.io.votable.tree.SimpleElement
# py:class astropy.io.votable.tree.SimpleElementWithContent
#
# Uncomment the following lines to enable the exceptions:
#
# for line in open('nitpick-exceptions'):
# if line.strip() == "" or line.startswith("#"):
# continue
# dtype, target = line.split(None, 1)
# target = target.strip()
# nitpick_ignore.append((dtype, six.u(target)))
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why is this here?

Copy link
Author

Choose a reason for hiding this comment

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

It is for in case we will need to use it in the future to turn on nitpicky. I have used it for some astropy docs before, and thought we might need it someday if our docs grow larger in size and there might be some problems that can only be resolved by nitpick-exceptions.

docs/contributing.rst Outdated Show resolved Hide resolved
docs/contributing.rst Outdated Show resolved Hide resolved
docs/contributing.rst Outdated Show resolved Hide resolved
docs/contributing.rst Outdated Show resolved Hide resolved
tox.ini Outdated Show resolved Hide resolved
Copy link
Collaborator

@nabobalis nabobalis left a comment

Choose a reason for hiding this comment

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

A few very minor changes.

Co-authored-by: Nabil Freij <nabil.freij@gmail.com>
@kakirastern
Copy link
Author

Thanks @nabobalis! Suggested changes committed and waiting for tests to pass.

Co-authored-by: Nabil Freij <nabil.freij@gmail.com>
@nabobalis nabobalis merged commit 2be3cdc into glue-viz:master Aug 10, 2020
@kakirastern
Copy link
Author

Appreciate it @nabobalis

@kakirastern kakirastern deleted the add-other-docs branch August 10, 2020 22:02
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.

2 participants