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.

Sign up for GitHub

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

Jump to bottom

APE N 🔥 experiments #83

Closed
wants to merge 40 commits into from
Closed

APE N 🔥 experiments #83

wants to merge 40 commits into from

Conversation

astrofrog
Copy link
Owner

@astrofrog astrofrog commented Feb 10, 2019 •
edited
Loading

  • Move data definition to package_data in setup.cfg - this will also involve tidying up the data so that it's in a few predictable directory names (e.g. always data/)

Cadair
Cadair reviewed Mar 7, 2019
View reviewed changes
setup.cfg Outdated
@@ -97,3 +108,7 @@ multi_line_output = 0
balanced_wrapping = True
include_trailing_comma = false
length_sort = True

[egg_info]
Copy link
Collaborator

Choose a reason for hiding this comment

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

fwiw, the existence of this section is incompatible with setuptools_scm

Copy link
Owner Author

Choose a reason for hiding this comment

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

I don't know where that came from...

@astrofrog astrofrog mentioned this pull request Aug 8, 2019
Merged
@bsipocz
Copy link

bsipocz commented Aug 8, 2019

nice title 🔥 😆

@astrofrog
Remove astropy-helpers submodule and use pyproject.toml instead
483f577
@astrofrog
Don't define setup_requires, and require Cython and jinja2 in pyproje…
37092f4 
…ct.toml
@astrofrog
Add pyproject.toml to manifest
8cd85dc
@astrofrog
Declare all package data in setup.cfg
e982fd4
@astrofrog
Remove unneeded setup_package.py files
3bd1ea1
@astrofrog
Switch to using extension-helpers
d6e83dc
@astrofrog
More fixes
16a3847
@astrofrog
Run preprocessing step when returning extensions
d06ceca
@astrofrog
Don't use 'numpy' alias in include_dirs
b632115
@astrofrog
TMP: simple Travis configuration with tox
63c28e4
@astrofrog
Use pyargs
a97b98b
@astrofrog
Use extras in tox config
87a2137
@astrofrog
Install 'all' for docs
ffbaeb4
@astrofrog
Remove conftest.py otherwise it causes pytest to local import astropy
5ef621f
@astrofrog
Run tests inside a temporary directory in tox
e68f7f5
@astrofrog
Fixed inclusion of data in setup.cfg
9ab63ab
@astrofrog
Increase verbosity on Travis
90143f7
@astrofrog
Don't change directory for docs
37595ef
@astrofrog
Split Travis job over four builds
09532cb
@astrofrog
Add pytest plugins via entry points
4d5c820
@astrofrog
Remove old uses of astropy.version
2c5b4d5
@astrofrog
Use setuptools_scm
f2f74ab
@astrofrog
Install graphviz on Travis
5b1f185
@astrofrog
Fix data paths
fb97087
@astrofrog astrofrog changed the base branch from master to 32-bit-3.5-3.7 November 28, 2019 15:07
@astrofrog astrofrog changed the base branch from 32-bit-3.5-3.7 to master November 28, 2019 15:07
Cadair
Cadair reviewed Nov 29, 2019
View reviewed changes
.travis.yml
env: TOXENV=py37-test
- python: 3.8
env: TOXENV=py38-test
- python: 3.6
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do you really need three different doc builds?

Copy link
Owner Author

Choose a reason for hiding this comment

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

No just testing out :)

@astrofrog
Copy link
Owner Author

Closing since the main PR against astropy core is open!

@astrofrog astrofrog closed this Jan 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Cadair Cadair Cadair left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@astrofrog @bsipocz @Cadair