Skip to content
Commisery

Update regex matching to handle proper class escaping #10

Sign in to view logs

Update regex matching to handle proper class escaping

Update regex matching to handle proper class escaping #10

Triggered via pull request April 6, 2024 19:02
@warthog9warthog9
synchronize #19
warthog9:update-regex
Status Failure
Total duration 16s
Artifacts –

commisery.yml

on: pull_request
commit-message  /  commit-message
7s
commit-message / commit-message
Fit to window
Zoom out
Zoom in

Annotations

2 errors
commit-message / commit-message
1 of the pull request's commits are not valid Conventional Commits
(Commit 6ad7901d) fix: Update regex matching to handle proper class escaping On Python 3.12 it's throwing compile warnings on the regex compiles: ``` spdx/verify-spdx-headers:8: SyntaxWarning: invalid escape sequence '\s' SPDX = re.compile(f'SPDX-License-Identifier:\s+({SLUG.pattern})') spdx/verify-spdx-headers:21: SyntaxWarning: invalid escape sequence '\s' pattern = f"^{init}\s*{SPDX.pattern}\s*{fini}\\s*$" spdx/verify-spdx-headers:21: SyntaxWarning: invalid escape sequence '\s' pattern = f"^{init}\s*{SPDX.pattern}\s*{fini}\s*$" ``` This escapes the the '\s' to be '\\s' instead as that is compliant, seems to compile correctly, and makes the warning go away. Quick checking on my own repos seemed to not break anything. Signed-off-by: John 'Warthog9' Hawley <jhawley@tenstorrent.com>
[C003] Description should not start with a capital case letter