Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

editorconfig: add max_line_length for Python files #13228

Merged

Conversation

sumnerevans
Copy link
Contributor

@sumnerevans sumnerevans commented Jul 8, 2022

See the documentation for the property here:
https://github.com/editorconfig/editorconfig/wiki/EditorConfig-Properties#max_line_length

Signed-off-by: Sumner Evans me@sumnerevans.com

Pull Request Checklist

  • Pull request is based on the develop branch
  • Pull request includes a changelog file. The entry should:
    • Be a short description of your change which makes sense to users. "Fixed a bug that prevented receiving messages from other servers." instead of "Moved X method from EventStore to EventWorkerStore.".
    • Use markdown where necessary, mostly for code blocks.
    • End with either a period (.) or an exclamation mark (!).
    • Start with a capital letter.
    • Feel free to credit yourself, by adding a sentence "Contributed by @github_username." or "Contributed by [Your Name]." to the end of the entry.
  • Pull request includes a sign off
  • Code style is correct
    (run the linters)

@sumnerevans sumnerevans requested a review from a team as a code owner July 8, 2022 16:08
Copy link
Contributor

@DMRobertson DMRobertson left a comment

Choose a reason for hiding this comment

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

Good idea!

See the documentation for the property here:
https://github.com/editorconfig/editorconfig/wiki/EditorConfig-Properties#max_line_length

Signed-off-by: Sumner Evans <me@sumnerevans.com>
@sumnerevans sumnerevans force-pushed the editorconfig-max-line-length branch 2 times, most recently from e3f9fb7 to 76e0cd8 Compare July 8, 2022 16:10
@DMRobertson DMRobertson enabled auto-merge (squash) July 8, 2022 16:11
@DMRobertson DMRobertson merged commit 739adf1 into matrix-org:develop Jul 8, 2022
@sumnerevans sumnerevans deleted the editorconfig-max-line-length branch July 8, 2022 17:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants