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

add task for simple auto fix #775

Merged
merged 1 commit into from
Oct 29, 2015
Merged

add task for simple auto fix #775

merged 1 commit into from
Oct 29, 2015

Conversation

erwinmombay
Copy link
Member

can fix:
semi
eqeqeq
quotes

@erwinmombay
Copy link
Member Author

can fix:
no-multi-spaces
array-bracket-spacing
computed-property-spacing
eol-last
indent
no-spaced-func
no-trailing-spaces
object-curly-spacing
space-after-keywords
space-before-blocks
space-before-function-paren
space-before-keywords
space-infix-ops
space-return-throw-case
space-unary-ops
arrow-spacing
generator-star-spacing

@erwinmombay
Copy link
Member Author

will add more autofixes at a later PR

@erwinmombay
Copy link
Member Author

@cramforce PTAL. would like to get this in so I can get the other lint autofix PR's through later.

@erwinmombay
add task for simple auto fix
b7b6f64 
can fix:
semi
eqeqeq
quotes
@cramforce
Copy link
Member

LGTM

erwinmombay added a commit that referenced this pull request Oct 29, 2015
@erwinmombay
Merge pull request #775 from erwinmombay/eslint
f0d81ba 
add task for simple auto fix
@erwinmombay erwinmombay merged commit f0d81ba into ampproject:master Oct 29, 2015
@qidonna qidonna mentioned this pull request Nov 11, 2019
Merged
@uditalias uditalias mentioned this pull request Nov 4, 2022
Open
@Elisabeth1983 Elisabeth1983 mentioned this pull request Nov 4, 2022
Open
@Masterci84 Masterci84 mentioned this pull request Nov 5, 2022
Open
@muratkalenderoglu muratkalenderoglu mentioned this pull request Nov 5, 2022
Open
@terrorizer1980 terrorizer1980 mentioned this pull request Nov 5, 2022
Closed
@shmoop207 shmoop207 mentioned this pull request Nov 15, 2022
Open
@Elisabeth1983 Elisabeth1983 mentioned this pull request Nov 15, 2022
Open
@Masterci84 Masterci84 mentioned this pull request Nov 16, 2022
Open
@muratkalenderoglu muratkalenderoglu mentioned this pull request Nov 16, 2022
Open
@terrorizer1980 terrorizer1980 mentioned this pull request Nov 16, 2022
Closed
@shmoop207 shmoop207 mentioned this pull request Nov 28, 2022
Open
@Elisabeth1983 Elisabeth1983 mentioned this pull request Nov 28, 2022
Open
@Masterci84 Masterci84 mentioned this pull request Nov 28, 2022
Open
@muratkalenderoglu muratkalenderoglu mentioned this pull request Nov 29, 2022
Open
@terrorizer1980 terrorizer1980 mentioned this pull request Nov 29, 2022
Closed
@Elisabeth1983 Elisabeth1983 mentioned this pull request Dec 25, 2022
Open
@shmoop207 shmoop207 mentioned this pull request Dec 25, 2022
Open
@Elisabeth1983 Elisabeth1983 mentioned this pull request Dec 25, 2022
Open
@Masterci84 Masterci84 mentioned this pull request Dec 25, 2022
Open
@muratkalenderoglu muratkalenderoglu mentioned this pull request Dec 26, 2022
Open
@terrorizer1980 terrorizer1980 mentioned this pull request Dec 26, 2022
Closed
@Elisabeth1983 Elisabeth1983 mentioned this pull request Oct 14, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@cramforce cramforce

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@erwinmombay @cramforce