Skip to content

Remove unused imports#180

Open
katesalazar wants to merge 1 commit intobitcoin-core:mainfrom
katesalazar:20250321
Open

Remove unused imports#180
katesalazar wants to merge 1 commit intobitcoin-core:mainfrom
katesalazar:20250321

Conversation

@katesalazar
Copy link
Copy Markdown

.gitignore: Ignore PyCharm metadata.
clang-format.py: Python 3.
list-pulls.py: Remove unused import.
termlib/attr.py: Explicit class methods.
termlib/tableprinter.py: Adapts signature change.
treehash512.py: Remove unused imports.

Comment thread .gitignore Outdated
Comment thread clang-format.py Outdated
Comment thread clang-format.py Outdated
@fanquake
Copy link
Copy Markdown
Member

Why are you changing the permissions of clang-format.py?

@katesalazar

This comment was marked as off-topic.

list-pulls.py: Remove unused import.
termlib/attr.py: Explicit class methods.
termlib/tableprinter.py: Adapts signature change.
treehash512.py: Remove unused imports.
Comment thread treehash512.py
@@ -1,5 +1,5 @@
#!/usr/bin/env python3
# Copyright (c) 2017 The Bitcoin Core developers
# Copyright (c) 2017, 2025 The Bitcoin Core developers
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
# Copyright (c) 2017, 2025 The Bitcoin Core developers
# Copyright (c) 2017-present The Bitcoin Core developers

nit: If this is touched, might as well remove the years, or just append -present, to avoid having to re-touch this again in the future for style fixups.

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.

4 participants