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

Remove all mention of Python 2.6.x support #27

Closed
mperrin opened this issue Aug 28, 2018 · 5 comments
Closed

Remove all mention of Python 2.6.x support #27

mperrin opened this issue Aug 28, 2018 · 5 comments

Comments

@mperrin
Copy link
Collaborator

mperrin commented Aug 28, 2018

Issue by josePhoenix
Monday Nov 24, 2014 at 21:31 GMT
Originally opened as mperrin/webbpsf#27


Since we don't actually have Python 2.6 support any more, we can amend the docs and get rid of a function in webbpsf.tkgui.


josePhoenix included the following code: https://github.com/mperrin/webbpsf/pull/27/commits

@mperrin
Copy link
Collaborator Author

mperrin commented Aug 28, 2018

Comment by josePhoenix
Monday Nov 24, 2014 at 21:31 GMT


Addresses #21.

@mperrin
Copy link
Collaborator Author

mperrin commented Aug 28, 2018

Comment by josePhoenix
Wednesday Dec 10, 2014 at 00:35 GMT


@mperrin can I merge this?

@mperrin
Copy link
Collaborator Author

mperrin commented Aug 28, 2018

Comment by mperrin
Wednesday Dec 10, 2014 at 15:29 GMT


The pull request appears to have accidentally picked up two binary swap files:
webbpsf/.OpticalTrainMW.py.swp
webbpsf/.trainDriverMW.py.swp

If you could please take those out, then the rest of this PR looks good to merge.

@mperrin
Copy link
Collaborator Author

mperrin commented Aug 28, 2018

Comment by josePhoenix
Wednesday Dec 10, 2014 at 15:32 GMT


Those files were in the repository already (see https://github.com/mperrin/webbpsf/tree/master/webbpsf). I just removed them while I was cleaning up.

@mperrin
Copy link
Collaborator Author

mperrin commented Aug 28, 2018

Comment by mperrin
Wednesday Dec 10, 2014 at 15:35 GMT


Oh whoops, I misread that those were additions rather than removals. Great, thanks.

@mperrin mperrin closed this as completed Aug 28, 2018
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

1 participant