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

DM-13997: {{Add template doc/ infrastructure}} #52

Merged
merged 1 commit into from Apr 17, 2018
Merged

Conversation

ebellm
Copy link
Contributor

@ebellm ebellm commented Apr 4, 2018

No description provided.

Copy link
Member

@jonathansick jonathansick left a comment

Choose a reason for hiding this comment

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

There's a mix-up between ip_diffim and ip_isr in manifest.yaml, but other than that it's good to go.


# Name of this package
# Also the name of the package documentation subdirectory.
package: "ip_diffim"
Copy link
Member

Choose a reason for hiding this comment

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

ip_isr

# List of names of Python modules in this package.
# For each module there is a corresponding module doc subdirectory.
modules:
- "lsst.ip.diffim"
Copy link
Member

Choose a reason for hiding this comment

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

lsst.ip.isr

# Name of the static content directories (subdirectories of `_static`).
# Static content directories are usually named after the package.
statics:
- "_static/ip_diffim"
Copy link
Member

Choose a reason for hiding this comment

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

ip_isr

@ebellm
Copy link
Contributor Author

ebellm commented Apr 17, 2018

Fixed, sorry about that. Can you give it the green light?

Copy link
Member

@jonathansick jonathansick left a comment

Choose a reason for hiding this comment

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

Looks good now 👍

@ebellm ebellm merged commit 1e2f31b into master Apr 17, 2018
@ktlim ktlim deleted the tickets/DM-13997 branch August 25, 2018 06:44
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.

None yet

2 participants