Skip to content

all: ruff --fix W605 Invalid_escape_sequences #10993

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
May 2, 2023
Merged

all: ruff --fix W605 Invalid_escape_sequences #10993

merged 1 commit into from
May 2, 2023

Conversation

cclauss
Copy link
Contributor

@cclauss cclauss commented Mar 10, 2023 •
edited
Loading

% ruff --select=W605 --exclude=tests --fix .

Found 17 errors (17 fixed, 0 remaining).

% ruff rule W605

invalid-escape-sequence (W605)

Derived from the pycodestyle linter.

Autofix is always available.

Message formats:

  • Invalid escape sequence: \{char}

These will also appear as warnings in pytest.
https://beta.ruff.rs

@github-actions
Copy link

Code size report:

   bare-arm:    +0 +0.000% 
minimal x86:    +0 +0.000% 
   unix x64:    +0 +0.000% standard
      stm32:    +0 +0.000% PYBV10
        rp2:    +0 +0.000% PICO

@codecov-commenter
Copy link

Codecov Report

Merging #10993 (dac1bbe) into master (4376c96) will not change coverage.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@           Coverage Diff           @@
##           master   #10993   +/-   ##
=======================================
  Coverage   98.50%   98.50%           
=======================================
  Files         155      155           
  Lines       20549    20549           
=======================================
  Hits        20241    20241           
  Misses        308      308

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@cclauss cclauss mentioned this pull request Mar 10, 2023
Merged
@cclauss cclauss force-pushed the ruff_--fix_W605_Invalid_escape_sequence branch from dac1bbe to f26e6eb Compare March 10, 2023 09:38
@cclauss cclauss changed the title ruff --fix W605 Invalid_escape_sequences all: ruff --fix W605 Invalid_escape_sequences Mar 10, 2023
@cclauss cclauss force-pushed the ruff_--fix_W605_Invalid_escape_sequence branch 2 times, most recently from 7d8c263 to c3f1384 Compare March 10, 2023 10:18
@dpgeorge dpgeorge force-pushed the ruff_--fix_W605_Invalid_escape_sequence branch from f197706 to 8f8bd98 Compare May 2, 2023 01:58
@dpgeorge dpgeorge merged commit 8f8bd98 into micropython:master May 2, 2023
@dpgeorge
Copy link
Member

dpgeorge commented May 2, 2023

Thanks!

@cclauss cclauss deleted the ruff_--fix_W605_Invalid_escape_sequence branch May 2, 2023 03:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@cclauss @codecov-commenter @dpgeorge