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

[MNT] Black codebase #584

Merged
merged 12 commits into from
Apr 4, 2024
Merged

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.

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
19 of 20 checks passed
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