Skip to content

chore(CI): Add row lenght - #30

Merged
mirzak merged 1 commit into
mainfrom
pyproject_lengh
Oct 23, 2025
Merged

chore(CI): Add row lenght#30
mirzak merged 1 commit into
mainfrom
pyproject_lengh

Conversation

@Dexter9532

Copy link
Copy Markdown
Contributor

did this because of row being to short.

@Dexter9532
Dexter9532 requested a review from mirzak October 22, 2025 13:27

@mirzak mirzak left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

There is a typo in your commit title,

chore(CI): Add row lengh

While at it, I would write something like:

chore(ruff): Update maximum line length to 120 columns

did this because of row being to short.

Would also update above to:

The default maximum line length is 80 columns, which is rather aggressive with modern tools.

@mirzak mirzak left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The commit message looks odd:

 chore(CI): chore(ruff): Update maximum line length

to 120 columns

The default maximum line length is 80 columns, which is
rather aggressive with modern tools.

Signed-off-by: Bobo Bäck Engström <bobo@id8-engineering.io>

You have an unexpected line break in the title, and you have chore(CI): chore(ruff):. Just chore(ruff) is enough.

The default maximum line length is 80 columns, which is
rather aggressive with modern tools.

Signed-off-by: Bobo Bäck Engström <bobo@id8-engineering.io>
@Dexter9532
Dexter9532 requested a review from mirzak October 23, 2025 09:47
@mirzak mirzak changed the title chore(CI): Add row lengh chore(CI): Add row lenght Oct 23, 2025
@mirzak
mirzak merged commit 40f1d70 into main Oct 23, 2025
20 checks passed
@mirzak
mirzak deleted the pyproject_lengh branch October 23, 2025 10:02
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