Skip to content

MNT: Temporarily ignore setuptools warning about distutils#10576

Merged
pllim merged 2 commits intoastropy:masterfrom
pllim:whitelist-setuptools
Jul 17, 2020
Merged

MNT: Temporarily ignore setuptools warning about distutils#10576
pllim merged 2 commits intoastropy:masterfrom
pllim:whitelist-setuptools

Conversation

@pllim
Copy link
Member

@pllim pllim commented Jul 17, 2020

Hopefully #10571 or its replacement will undo this later. This is a temporarily bandaid to allow CI to be green again until we find a proper solution.

@pllim
Copy link
Member Author

pllim commented Jul 17, 2020

RTD warnings are unrelated but I'll try to patch them here too.

@pllim
Copy link
Member Author

pllim commented Jul 17, 2020

CI green again. Remote failure unrelated.

Copy link
Member

@mwcraig mwcraig left a comment

Choose a reason for hiding this comment

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

This seems like a reasonable approach for now. I'll return to #10571 in an hour or so but this at least gets things moving again.

@pllim pllim merged commit d10d0f2 into astropy:master Jul 17, 2020
@pllim pllim deleted the whitelist-setuptools branch July 17, 2020 17:53
@pllim pllim modified the milestones: v4.2, v4.0.2 Jul 17, 2020
@pllim
Copy link
Member Author

pllim commented Jul 17, 2020

Actually we might want this in LTS, so I changed the milestone. Feel free to change it back if we don't.

@mhvk
Copy link
Contributor

mhvk commented Jul 17, 2020

Great that tests pass again, thanks both!

eteq pushed a commit that referenced this pull request Jul 28, 2020
MNT: Temporarily ignore setuptools warning about distutils
eteq pushed a commit that referenced this pull request Jul 28, 2020
MNT: Temporarily ignore setuptools warning about distutils
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants