Skip to content

fix: linting issues#166

Merged
kdeldycke merged 2 commits intokdeldycke:mainfrom
emmanuelgautier:fix/lint-issues
May 21, 2025
Merged

fix: linting issues#166
kdeldycke merged 2 commits intokdeldycke:mainfrom
emmanuelgautier:fix/lint-issues

Conversation

@emmanuelgautier
Copy link
Contributor

@emmanuelgautier emmanuelgautier commented May 14, 2025

Motivation

There were some linting issues. This PR aims to fix some of them.

image

Affiliation

  • I am the author of the article or project
  • I am working for/with the company which is publishing the article or project
  • I'm just a rando who stumbled upon this via social networks

Self checks

@kdeldycke
Copy link
Owner

Thanks for the PR! I am ready to merge it, but can you also apply these changes to the chinese version in readme.zh.md file please? 🙏

@kdeldycke kdeldycke added 🩹 fix link Existing link need to be updated 📚 documentation Update to non-code (readme, docstrings, typos, …) and removed 🩹 fix link Existing link need to be updated labels May 15, 2025
@emmanuelgautier
Copy link
Contributor Author

I've applied the same changes to the Chinese version in the readme.zh.md file as requested.

That said, I'm not very familiar with the language, so I might not be able to contribute much more to that version. But happy to help however I can!

@kdeldycke kdeldycke merged commit 282a19f into kdeldycke:main May 21, 2025
@kdeldycke
Copy link
Owner

Thanks for the update! Let's merge! :)

@emmanuelgautier emmanuelgautier deleted the fix/lint-issues branch May 26, 2025 12:18
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 25, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

📚 documentation Update to non-code (readme, docstrings, typos, …)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants