这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@LyonJack
Copy link
Contributor

@LyonJack LyonJack commented Jul 3, 2020

No description provided.

@Dirkster99 Dirkster99 merged commit 562ba9f into Dirkster99:master Jul 3, 2020
@Dirkster99
Copy link
Owner

I think we should try to use CodeMaid - it looks pretty cool to me :-) Do you know how we can setup and store a settings file in the project with rules such that the current order of items (in most classes) remains?

-> I prefer this order of regions in each class:

  • fields
  • constructors
  • events
  • properties
  • methods
  • private classes

-> Members in each region should be sorted by accessability:
static public, public, static internal, internal, static protected, protected, static private, private

(but it would be OK to have exceptions with dependency properties for example)

-> Public and Protected Methods, Properties, Events should always be documented

@LyonJack
Copy link
Contributor Author

LyonJack commented Jul 8, 2020

I think we should try to use CodeMaid - it looks pretty cool to me :-) Do you know how we can setup and store a settings file in the project with rules such that the current order of items (in most classes) remains?

-> I prefer this order of regions in each class:

  • fields
  • constructors
  • events
  • properties
  • methods
  • private classes

-> Members in each region should be sorted by accessability:
static public, public, static internal, internal, static protected, protected, static private, private

(but it would be OK to have exceptions with dependency properties for example)

-> Public and Protected Methods, Properties, Events should always be documented

I have add a default config about codemaid(CodeMaid.config),look at my pull request,you can swtich to special config in the option of codemaid

Dirkster99 added a commit that referenced this pull request Jul 8, 2020
add a default config of CodeMaid.config PR #172
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.

2 participants