Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Event handling method parameter names must follow guideline #3

Closed
RalfKoban opened this issue Jan 29, 2018 · 0 comments
Closed

Event handling method parameter names must follow guideline #3

RalfKoban opened this issue Jan 29, 2018 · 0 comments
Labels
Area: analyzer Area: naming feature

Comments

@RalfKoban
Copy link
Owner

Event handling methods shall have the following signature:
private void OnXyzEvent(object sender, XyzEventArgs e)

  1. The 1st parameter shall be named sender.
  2. The 2nd parameter shall be named e.
@RalfKoban RalfKoban changed the title Analyzer for naming of parameters of method signature of event handling method Analyzer for naming of event handling method parameters Jan 29, 2018
@RalfKoban RalfKoban changed the title Analyzer for naming of event handling method parameters Event handling method parameter names must follow guideline Jan 29, 2018
@RalfKoban RalfKoban added backlog and removed backlog labels Feb 4, 2018
@RalfKoban RalfKoban reopened this Feb 4, 2018
RalfKoban added a commit that referenced this issue Feb 5, 2018
@RalfKoban
fix for issue #16 for specific situation where issue #3 applies at well
a119759
RalfKoban added a commit that referenced this issue Feb 9, 2018
@RalfKoban
Refactorings (reorderings for #3, #5, #12, #16 and #26)
336e98b
@RalfKoban RalfKoban mentioned this issue Feb 19, 2018
Closed
@RalfKoban RalfKoban removed the review label Feb 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Area: analyzer Area: naming feature
Projects
MiKo Analyzers
Status: Done
Milestone
No milestone
Development

No branches or pull requests
1 participant
@RalfKoban