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 python 2 from build and CI scripts #1583

Merged
merged 8 commits into from
Oct 16, 2021
Merged

Remove python 2 from build and CI scripts #1583

merged 8 commits into from
Oct 16, 2021

Conversation

timkpaine
Copy link
Member

@timkpaine timkpaine commented Oct 14, 2021

I don't care about python 2 anymore, and I was assured that nobody else would after December 2020 😉 🇬🇷 🦉

I don't care about python 2 anymore, and [I was repeatedly assured time and time again that nobody else would after December 2020](https://www.efinancialcareers.com/news/2021/02/jpmorgan-still-has-its-python-2-issue) 😉 🇬🇷 🦉
@texodus
Copy link
Member

texodus commented Oct 14, 2021

Perspective hasn't supported Python2 for a long time #1336, no assurances necessary, even from an esteemed pillar of technology journalism like ... (checks notes) ... efinancialcareers.com.

I would like to keep https://github.com/finos/perspective/pull/1583/files#diff-7915b9b726a397ae7ba6af7b9703633d21c031ebf21682f3ee7e6a4ec52837a5L356 though, but a working version that does not cause weekly all-in CI builds to fail.

@texodus texodus removed the breaking label Oct 14, 2021
@timkpaine
Copy link
Member Author

I would like to keep https://github.com/finos/perspective/pull/1583/files#diff-7915b9b726a397ae7ba6af7b9703633d21c031ebf21682f3ee7e6a4ec52837a5L356 though, but a working version that does not cause weekly all-in CI builds to fail.

re-added, should be fine with #1563

@timkpaine timkpaine changed the title Remove python 2 support Remove python 2 from builds scripts Oct 14, 2021
@timkpaine timkpaine changed the title Remove python 2 from builds scripts Remove python 2 from build and CI scripts Oct 14, 2021
@texodus
Copy link
Member

texodus commented Oct 16, 2021

Looks good!

@texodus texodus merged commit ab5ed59 into master Oct 16, 2021
@texodus texodus deleted the timkpaine-patch-1 branch October 16, 2021 00:14
@texodus texodus added the internal Internal refactoring and code quality improvement label Nov 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internal Internal refactoring and code quality improvement Python
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants