Skip to content

[IMPROVE] Use eslint-config package#12044

Merged
ggazzo merged 1 commit intodevelopfrom
use-eslint-package
Sep 14, 2018
Merged

[IMPROVE] Use eslint-config package#12044
ggazzo merged 1 commit intodevelopfrom
use-eslint-package

Conversation

@sampaiodiego
Copy link
Copy Markdown
Member

I moved all eslint rules to a new package called @rocket.chat/eslint-config.

This will help us using the same eslint rules across all projects.

I decided to keep the same rules for now, but I already have a list of a few rules I would like to change. 😬

Copy link
Copy Markdown
Member

@ggazzo ggazzo left a comment

Choose a reason for hiding this comment

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

Amazing!

@ggazzo ggazzo merged commit b9383ab into develop Sep 14, 2018
@ggazzo ggazzo deleted the use-eslint-package branch September 14, 2018 18:38
engelgabriel added a commit that referenced this pull request Sep 19, 2018
* develop: (27 commits)
  Fix stylelint
  [FIX]  Login error message not obvious if user not activated (#11785)
  [NEW] REST endpoints to get moderators from groups and channels (#11909)
  [NEW] User preference for 24- or 12-hour clock (#11169)
  [FIX] Adding scroll bar to read receipts modal (#11919)
  [FIX] Fixing translation on 'yesterday' word when calling timeAgo function (#11946)
  [FIX] Fixing spacement between tags and words on some labels (#12018)
  [FIX] video message recording, issue #11651 (#12031)
  [FIX] Prevent form submission in Files List search (#11999)
  [NEW] REST endpoint to set groups' announcement (#11905)
  [FIX] Re-add the eye-off icon (#12079)
  [NEW] Livechat trigger option to run only once (#12068)
  [IMPROVE] Use eslint-config package (#12044)
  [FIX] Internal error when cross-origin with CORS is disabled (#11953)
  [FIX] Message reaction in GraphQL API (#11967)
  [FIX] Direct messages leaking into logs (#11863)
  Fix the stylelint (#11991)
  [NEW] REST endpoints to create roles and assign roles to users (#11855)
  [FIX] Hidden admin sidenav on embedded layout (#12025)
  [FIX] Update user information not possible by admin if disabled to users (#11955)
  ...

# Conflicts:
#	package-lock.json
#	package.json
This was referenced Sep 28, 2018
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.

3 participants