Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Sep 18, 2023. It is now read-only.

Add no-method-prefixed-with-on rule #24

Merged
merged 1 commit into from
Mar 30, 2021
Merged

Add no-method-prefixed-with-on rule #24

merged 1 commit into from
Mar 30, 2021

Conversation

koddsson
Copy link
Contributor

@koddsson koddsson commented Mar 30, 2021 •
edited
Loading

This rule disallows any method names that start with on in a Custom Element class definition.

Ref: #3

@koddsson koddsson force-pushed the no-method-prefixed-with-on branch from 71c30ac to 3e4f168 Compare March 30, 2021 12:28
@koddsson koddsson mentioned this pull request Mar 30, 2021
Closed
16 tasks
keithamus
keithamus approved these changes Mar 30, 2021
View reviewed changes
Copy link
Contributor

@keithamus keithamus left a comment

Choose a reason for hiding this comment

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

Looks great!

@koddsson koddsson merged commit 9c3ad37 into main Mar 30, 2021
@koddsson koddsson deleted the no-method-prefixed-with-on branch April 9, 2021 11:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@keithamus keithamus keithamus approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@koddsson @keithamus