Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Rule Request: [order_of_properties] #2410

Open
2 tasks done
AndriiPetrovDev opened this issue Sep 18, 2018 · 5 comments · May be fixed by #3932
Open
2 tasks done

Rule Request: [order_of_properties] #2410

AndriiPetrovDev opened this issue Sep 18, 2018 · 5 comments · May be fixed by #3932
Labels
rule-request Requests for a new rules.

Comments

@AndriiPetrovDev
Copy link

AndriiPetrovDev commented Sep 18, 2018

New Issue Checklist

New rule request

Please describe the rule idea, format
this issue's title as Rule Request: [Rule Name] and describe:

  1. Why should this rule be added? Share links to existing discussion about what
    the community thinks about this.
  2. Provide several examples of what would and wouldn't trigger violations.
  3. Should the rule be configurable, if so what parameters should be configurable?
  4. Should the rule be opt-in or enabled by default? Why?
    See README.md for guidelines on when to mark a rule as opt-in.

I need rule to control properties groups order to keep file neat
All properties should be divided on several groups in order:

public let
public var

internal let
internal var

private let
private var

IBOutlet private var

The rule should triggering on mixing up this groups and on incorrect order
The rule should be configurable, I need to set desired order, or maybe order written above will bw enough
The rule should be opt-in

@realm-probot realm-probot bot added the O:User label Sep 18, 2018
@AndriiPetrovDev AndriiPetrovDev changed the title [OrderOfProperties] Rule Request: [order_of_properties] Sep 18, 2018
@marcelofabri marcelofabri added the rule-request Requests for a new rules. label Sep 18, 2018
@kacper1703
Copy link

Any updates on that? Would love to have that too in my project.

@marcelofabri
Copy link
Collaborator

@kacper1703 feel free to open a PR! 💯

@piobyz
Copy link

piobyz commented May 6, 2019

This could be done on top of #2296 as suggested in #2699

@sarperdag
Copy link

Would really love to see this rule get implemented

@weakfl
Copy link

weakfl commented Aug 26, 2019

+1, this would be great

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
rule-request Requests for a new rules.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants