Skip to content

add more ruff lints#2061

Merged
jensens merged 17 commits intocookiecutter:mainfrom
danieleades:ruff-more-lints
Jul 9, 2024
Merged

add more ruff lints#2061
jensens merged 17 commits intocookiecutter:mainfrom
danieleades:ruff-more-lints

Conversation

@danieleades
Copy link
Copy Markdown
Contributor

No description provided.

@danieleades danieleades marked this pull request as ready for review April 7, 2024 19:59
@jensens jensens added the code style This issue/PR relates to the project code style modification label Apr 25, 2024
@jensens
Copy link
Copy Markdown
Contributor

jensens commented Apr 25, 2024

This one needs manual conflict resolution.

@danieleades
Copy link
Copy Markdown
Contributor Author

This one needs manual conflict resolution.

that should do it!

failing CI jobs due to #2063

@danieleades
Copy link
Copy Markdown
Contributor Author

the ubuntu 3.7 job will need to be removed as an 'expected check' in the repo config

logger.debug('Writing contents to file %s', outfile)

with open(outfile, 'w', encoding='utf-8', newline=newline) as fh:
with open(outfile, 'w', encoding='utf-8', newline=newline) as fh: # noqa: FURB103 (false positive for python < 3.10)
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@jensens jensens merged commit b445123 into cookiecutter:main Jul 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code style This issue/PR relates to the project code style modification

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants