Skip to content

How can we be happy if clippy is not?#1990

Merged
evanlinjin merged 3 commits intobitcoindevkit:masterfrom
evanlinjin:clippy
Jul 10, 2025
Merged

How can we be happy if clippy is not?#1990
evanlinjin merged 3 commits intobitcoindevkit:masterfrom
evanlinjin:clippy

Conversation

@evanlinjin
Copy link
Copy Markdown
Member

Description

I made clippy happy.

Checklists

All Submissions:

@evanlinjin evanlinjin merged commit a7c1bcc into bitcoindevkit:master Jul 10, 2025
19 checks passed
Comment thread .gitignore
*.db
*.sqlite*

crates/electrum/target
Copy link
Copy Markdown
Member

@notmandatory notmandatory Jul 10, 2025

Choose a reason for hiding this comment

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

Small nit but this line isn't needed. Can be taking out in one of the current open PRs.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Or **/target to ignore all target sub directories.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants