Skip to content

Conversation

@bruAristimunha
Copy link
Collaborator

No description provided.

@agramfort
Copy link
Collaborator

@bruAristimunha you should setup a precommit hook for this so black is run on the files when you commit.

@codecov
Copy link

codecov bot commented Apr 2, 2024

Codecov Report

Merging #584 (3d08589) into master (a8801c9) will not change coverage.
The diff coverage is 78.55%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #584   +/-   ##
=======================================
  Coverage   84.11%   84.11%           
=======================================
  Files          67       67           
  Lines        5419     5419           
=======================================
  Hits         4558     4558           
  Misses        861      861           

@bruAristimunha
Copy link
Collaborator Author

I added @agramfort! It should be run with ruff, but I don't know why it is not working, so I added an extra black hook for pre-commit.

@bruAristimunha bruAristimunha requested a review from agramfort April 2, 2024 16:10
@bruAristimunha bruAristimunha changed the title Black codebase [MNT] Black codebase Apr 2, 2024
@bruAristimunha
Copy link
Collaborator Author

Hi @agramfort,

I saw this mne PR 12261, and I changed here for ruff format, now we are using the ruff properly

@agramfort agramfort merged commit 9e459e9 into braindecode:master Apr 4, 2024
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