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

Add Connectome Mapper 3 #288

Merged
merged 6 commits into from Sep 25, 2020
Merged

Conversation

@sebastientourbier
Copy link
Contributor

@sebastientourbier sebastientourbier commented Sep 4, 2020

If you're adding a project to the Python 3 statement, please ensure you have:

  • Discussed dropping Python 2 support with all maintainers
  • Discussed signing the statement with all maintainers
  • Added a suitable logo if the project has one
    • Approximately square logos generally fit better than banners
    • Compressed the logo to save bandwidth (run scripts/squash-images.sh on it)
  • Checked the Netlify preview made from the pull request
…timeline
@takluyver
Copy link
Member

@takluyver takluyver commented Sep 8, 2020

Can you run scripts/squash-images.sh assets/cmp.png to compress the logo?

site.js Outdated
{content: 'v3.0.0-beta-x beta series (Py2 only)', start: '2019-04-24', end:'2020-08-24', py2:true},
{content: 'v3.x.y[-RCz] releases and release candidates (Py3 only)', start: '2020-08-24', end:'2023-12-31'},
Comment on lines 358 to 359

This comment has been minimized.

@takluyver

takluyver Sep 8, 2020
Member

This is pretty confusing. Your betas were Python 2 only but your release candidates onwards were Python 3 only? It might be better just to leave it out, or say that the 3.x series is Python 3 only.

@takluyver
Copy link
Member

@takluyver takluyver commented Sep 25, 2020

Thanks

@takluyver takluyver merged commit 8b61672 into python3statement:master Sep 25, 2020
1 check passed
1 check passed
deploy/netlify Deploy preview ready!
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.