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

MAINT Remove unsupported Python 2 notebook option #421

Merged

Conversation

stephen-hoover
Copy link

Civis Platform no longer supports Python 2 notebooks (currently Python 3 and R), but the CLI was still presenting Python 2 as an option.

Civis Platform no longer supports Python 2 notebooks (currently Python 3 and R), but the CLI was still presenting Python 2 as an option.
mattyb
mattyb previously approved these changes Apr 27, 2021
Copy link
Member

@mattyb mattyb left a comment

Choose a reason for hiding this comment

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

LGTM!

@mattyb
Copy link
Member

mattyb commented Apr 27, 2021

sorry, could you add an entry to the unreleased section of the CHANGELOG as well?

@stephen-hoover
Copy link
Author

Sorry about that. Done.

Copy link
Member

@mattyb mattyb left a comment

Choose a reason for hiding this comment

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

LGTM (again)!

@stephen-hoover stephen-hoover merged commit 75e3767 into civisanalytics:master Apr 27, 2021
@stephen-hoover stephen-hoover deleted the remove-python2-from-cli branch April 27, 2021 18:32
@jacksonlee-civis jacksonlee-civis added this to the v1.16.0 milestone Nov 29, 2021
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.

None yet

3 participants