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

chore: remove __future__ imports #61

Merged
merged 1 commit into from
May 30, 2024
Merged

chore: remove __future__ imports #61

merged 1 commit into from
May 30, 2024

Conversation

jsstevenson
Copy link
Contributor

@jsstevenson jsstevenson commented May 29, 2024

Doing my daily biocommons cleanup. None of these are necessary anymore: https://docs.python.org/3/library/__future__.html

  • as a side note, I'm always a little hesitant to edit stuff in bin/ and sbin/ for some reason -- I often wonder if we should be including these commands in the released package (i.e. as entry-points)

@jsstevenson jsstevenson requested a review from a team as a code owner May 29, 2024 18:26
@jsstevenson jsstevenson merged commit e707773 into main May 30, 2024
13 checks passed
@jsstevenson jsstevenson deleted the rm-future branch May 30, 2024 12:21
@reece
Copy link
Member

reece commented Jun 24, 2024

Most bin and sbin scripts are not required to use the package, so I would recommend leaving them as-is. I'm sure there are exceptions that merit inclusion in the package as entry points.

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.

3 participants