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.

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

Setup eslint #4

Merged
merged 5 commits into from
Oct 16, 2020
Merged

Setup eslint #4

merged 5 commits into from
Oct 16, 2020

Conversation

rogebrd
Copy link
Owner

@rogebrd rogebrd commented Oct 16, 2020

  • Setup eslint under Airbnb standards
  • Add CI action with linter
  • Update code to fit linting standards

Checklist

Is This a Code Change?

  • Non-code related change (markdown/git settings etc)
  • SDK Code Change
  • Example/Test Code Change

Validation

  • Does npm test pass?
  • Does npm run build pass?
  • Does npm run lint pass?

- Setup eslint under Airbnb standards
- Add CI action with linter
- Update code to fit linting standards
@rogebrd rogebrd merged commit eefb36e into main Oct 16, 2020
@rogebrd rogebrd deleted the add_linter branch October 16, 2020 22:51
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.

1 participant