Skip to content

Conversation

@clizbe
Copy link
Member

@clizbe clizbe commented Mar 22, 2024

(description of changes)

Fixes #517

Checklist before merging

  • Documentation is up-to-date
  • [NA] Unit tests have been added/updated accordingly
  • Code has been formatted according to SpineOpt's style
  • [NA] Unit tests pass

surround_whereop_typeparameters = true
trailing_comma = "nothing"
align_conditional = true
annotate_untyped_fields_with_any = true
Copy link
Collaborator

Choose a reason for hiding this comment

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

annotate_untyped_fields_with_any = true? I think this should be false.

trailing_comma = "nothing"? we could have true here I think.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah we can set untyped_fields to false.
For the trailing_comma, I set it to nothing because it seems we like using it in some cases and not in others, and I assumed there was a reason. But if there's no reason we can just enforce it. :)

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.

Use an automatic code formatter

2 participants