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

Cut back the default intersphinx mapping #9

Open
bsipocz opened this issue May 16, 2018 · 2 comments
Open

Cut back the default intersphinx mapping #9

bsipocz opened this issue May 16, 2018 · 2 comments

Comments

@bsipocz
Copy link
Member

bsipocz commented May 16, 2018

Actually I'm not sure why we provide the intersphinx mapping in here for so many optional dependencies but missed some others (e.g. why h5py but not pandas)?

I suggest to only keep python, numpy and astropy here, and the rest can go into docs/conf.py for the package repositories. If we keep it however, consider adding more mappings.

@astrofrog
Copy link
Member

I'd be more inclined to include pandas than remove existing ones. If we remove ones we'll probably have to create v2 of the default configuration, and I'd rather not create new configurations lightly.

@bsipocz
Copy link
Member Author

bsipocz commented May 20, 2018

I'm actually fine by adding more, pandas, ipython and pytest that core also uses. From those probably pandas is the only one useful for multiple packages. I would maybe consider adding scikit-image, too.

Also, at some point when we have the docs set up, I think it would make sense to have it there for all the astropy coordinated packages, too (though I'm not sure how sphinx behaves when it has the mapping of a released version for a package it's actually building).

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

No branches or pull requests

2 participants