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

DOC: Description and workaround of Spyder UMR conflict in known_issues #7375

Closed
wants to merge 1 commit into from

Conversation

pllim
Copy link
Member

@pllim pllim commented Apr 14, 2018

Fix #7363

Close #7374

This is basically the same as #7374 but it is opened against v2.0.x branch instead of master. I tried cherry-picking but that didn't work. I also didn't want to clobber @dhomeier 's branch although theoretically I could.

@astropy-bot
Copy link

astropy-bot bot commented Apr 14, 2018

Hi there @pllim 👋 - thanks for the pull request! I'm just a friendly 🤖 that checks for issues related to the changelog and making sure that this pull request is milestoned and labeled correctly. This is mainly intended for the maintainers, so if you are not a maintainer you can ignore this, and a maintainer will let you know if any action is required on your part 😃.

Everything looks good from my point of view! 👍

If there are any issues with this message, please report them here.

@pllim
Copy link
Member Author

pllim commented Apr 14, 2018

Huh... CircleCI failed with the following error. Can anyone try to have it to pip install future to fix that error? Or can we ignore it?

Traceback (most recent call last):
  File "setup.py", line 7, in <module>
    from astropy_helpers.setup_helpers import register_commands, get_package_info
  File "/root/project/astropy_helpers/astropy_helpers/__init__.py", line 46, in <module>
    import builtins
ImportError: No module named builtins

@pllim
Copy link
Member Author

pllim commented Apr 14, 2018

FWIW I don't think the failures are related to this PR. Perhaps they are fixed in master but just not backported yet. 🤷‍♀️

@pllim
Copy link
Member Author

pllim commented Apr 16, 2018

Looking at spyder-ide/spyder#6962, it seems that this is being addressed over there, so I am closing this without merge.

@pllim pllim closed this Apr 16, 2018
@pllim pllim deleted the spyder_umr branch April 16, 2018 14:33
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.

2 participants