Skip to content

Fixed missed inline import for subprocess.Popen#1785

Merged
embray merged 1 commit intoastropy:masterfrom
jwoillez:fix_setup_helpers_Popen
Nov 14, 2013
Merged

Fixed missed inline import for subprocess.Popen#1785
embray merged 1 commit intoastropy:masterfrom
jwoillez:fix_setup_helpers_Popen

Conversation

@jwoillez
Copy link
Member

The little bug was triggered by this build of astroquery: https://travis-ci.org/astropy/astroquery/jobs/13965996

@astrofrog
Copy link
Member

@embray @eteq - if possible I think we should get this in the final 0.3.0

@astrofrog
Copy link
Member

(but the RC has already been tagged, so we shouldn't delay that, let's just backport this to 0.3.x once the branch is created, then include it in the final version)

@eteq
Copy link
Member

eteq commented Nov 14, 2013

I agree with @astrofrog this should go into 0.3.0 - @embray, do you want to merge this yourself so you can backport it, or can we just merge it and you'll run the script before the final release?

@embray
Copy link
Member

embray commented Nov 14, 2013

Oops. This may well be my fault in fact. Yes, it's because of this sort of thing that we make release candidates :)

(This might be a good place to mention that the 0.3 release candidate is out and people should download it and use it: https://testpypi.python.org/pypi/astropy/0.3rc1)

embray added a commit that referenced this pull request Nov 14, 2013
Fixed missed inline import for subprocess.Popen
@embray embray merged commit 7a9abfd into astropy:master Nov 14, 2013
embray added a commit that referenced this pull request Nov 14, 2013
Fixed missed inline import for subprocess.Popen
@jwoillez jwoillez deleted the fix_setup_helpers_Popen branch November 18, 2013 09:33
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.

4 participants