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 a README with brief usage instructions and policy on adding packages #1

Merged
merged 1 commit into from
Mar 25, 2016

Conversation

mwcraig
Copy link
Contributor

@mwcraig mwcraig commented Mar 25, 2016

@eteq @Cadair @ngoldbaum please take a look

@mwcraig
Copy link
Contributor Author

mwcraig commented Mar 25, 2016

Comments still welcome, but merging to get some kind of README up...

@mwcraig mwcraig merged commit ad91e4e into OpenAstronomy:master Mar 25, 2016

+ `astropy`: channel for [astropy](http://astropy.org) [affiliated packages](http://astropy.org/affiliated)

## How can I add my package?
Copy link

Choose a reason for hiding this comment

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

Not strictly related to the README itself... but possibly. @mwcraig : does this automatically include the affiliated package channel? If so, perhaps this section should also include something along the lines of "if your package is or might be an astropy affiliated package, use that channel instead"?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@eteq At the moment the transfer is manual (in the sense that I ran a script locally to do the initial copying). I've been meaning to open a PR to conda-build-all to allow cross-channel copying, but it won't happen for a couple more weeks at least.

It is tempting to move all of the building over here because it has a separate travis/appveyor build queue, but that gets a little complicated for maintaining a separate affiliate channel.

Can you open an issue or PR for this?

Copy link

Choose a reason for hiding this comment

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

@mwcraig - will open an issue (as I don't think I have the expertise to do a PR!)

Copy link

@eteq eteq Apr 18, 2016

Choose a reason for hiding this comment

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

Done - see #10.

One item that's specifically about the README, though, @mwcraig : do you think it's reasonable to explicitly say that a way to get on this list (maybe even a "recommended" way) is to be astropy-affiliated? If so, I can make a PR to do that.

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