Skip to content

use line-length=120 for better code readability #213

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
Mar 23, 2023
Merged

use line-length=120 for better code readability #213

merged 1 commit into from
Mar 23, 2023

Conversation

Valeria1235
Copy link
Contributor

@Valeria1235 Valeria1235 commented Mar 14, 2023 •
edited
Loading

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Set line-length = 120

Pull request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

What is the current behavior?

Too many lines breaks into two and more lines

@Valeria1235 Valeria1235 force-pushed the black-up-line-length branch from 3eb40a5 to 01018f4 Compare March 14, 2023 15:26
@Valeria1235 Valeria1235 requested a review from rekby March 14, 2023 20:30
@Valeria1235 Valeria1235 marked this pull request as ready for review March 14, 2023 20:30
rekby
rekby previously approved these changes Mar 14, 2023
View reviewed changes
@rekby rekby self-requested a review March 14, 2023 22:49
@rekby rekby dismissed their stale review March 14, 2023 22:50

need wait before merge for prevent mass merge-conflicts

@Valeria1235 Valeria1235 force-pushed the black-up-line-length branch from bef067f to efb7218 Compare March 15, 2023 10:56
@Valeria1235 Valeria1235 force-pushed the black-up-line-length branch from efb7218 to 1835f6c Compare March 17, 2023 16:00
Base automatically changed from v3 to main March 22, 2023 15:31
@Valeria1235 Valeria1235 force-pushed the black-up-line-length branch 2 times, most recently from 0b1af44 to 0b215fe Compare March 22, 2023 16:22
@Valeria1235 Valeria1235 force-pushed the black-up-line-length branch from 0b215fe to ba6c081 Compare March 23, 2023 10:22
@Valeria1235 Valeria1235 merged commit a142e80 into main Mar 23, 2023
@Valeria1235 Valeria1235 deleted the black-up-line-length branch March 23, 2023 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rekby rekby rekby approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Valeria1235 @rekby