Skip to content

Conversation

@Orlyanskiy93
Copy link
Contributor

Запретить конформить ConfigurableView/ConfigurableCell non-final классам

#264

add final class rule for ConfigurableView and ConfigurableCell

final_class_for_configurable:
name: "Final class for ConfigurableView and ConfigurableCell"
regex: '(?<!final\s)(class\s\w*:(\s\w*,)?\s(ConfigurableView|ConfigurableCell))'
Copy link
Contributor

Choose a reason for hiding this comment

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

Screenshot 2021-08-16 at 12 48 39

возможно ли задетектить такой кейс?


final_class_for_configurable:
name: "Final class for ConfigurableView and ConfigurableCell"
regex: '(?<!final\s)(class\s\w*:(\s\w*,)?\s(ConfigurableView|ConfigurableCell))'
Copy link
Contributor

Choose a reason for hiding this comment

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

Screenshot 2021-08-16 at 12 50 30


final_class_for_configurable:
name: "Final class for ConfigurableView and ConfigurableCell"
regex: '(?<!final\s)(class\s\w*:(\s\w*,)?\s(ConfigurableView|ConfigurableCell))'
Copy link
Contributor

Choose a reason for hiding this comment

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

Screenshot 2021-08-16 at 12 51 21


final_class_for_configurable:
name: "Final class for ConfigurableView and ConfigurableCell"
regex: '(?<!final\s)((class|extention)\s.*:(\s\w*,)*?\s(ConfigurableView|ConfigurableCell))'
Copy link
Contributor

Choose a reason for hiding this comment

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

extension

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.

3 participants