Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Remove extra period#1001

Merged
PGijsbers merged 3 commits intodevelopopenml/openml-python:developfrom
PGijsbers-patch-1openml/openml-python:PGijsbers-patch-1Copy head branch name to clipboard
Nov 17, 2020
Merged

Remove extra period#1001
PGijsbers merged 3 commits intodevelopopenml/openml-python:developfrom
PGijsbers-patch-1openml/openml-python:PGijsbers-patch-1Copy head branch name to clipboard

Conversation

@PGijsbers
Copy link
Collaborator

@PGijsbers PGijsbers commented Nov 16, 2020

It is added by the Copyright template already (as observed from built docs).
Saw the old code for getting the year from the copyright message, and decided I might as well touch that up for readable python3.

It is added by the Copyright template already (as observed from built docs).
@PGijsbers PGijsbers requested a review from Neeratyoy November 16, 2020 12:56
u-strings are not needed in py3, directly getting the year from localtime is more readable
Required by Black
@PGijsbers PGijsbers merged commit 5d5a48e into develop Nov 17, 2020
@PGijsbers PGijsbers deleted the PGijsbers-patch-1 branch November 17, 2020 09:59
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.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.