Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

Remove linebreak-style Rule for ES6 #169

Merged
merged 3 commits into from
May 29, 2019
Merged

Remove linebreak-style Rule for ES6 #169

merged 3 commits into from
May 29, 2019

Conversation

bendvc
Copy link
Contributor

@bendvc bendvc commented May 22, 2019

Some developers have reported issues with their linter reporting errors for line endings. This is because we have our style set to unix. This works fine with unix obviously and macos, but it doens't play well with windows users unless they make changes to their IDE settings. Getting them to make those changes is probably more hassle than it's worth. So we have decided to remove this rule.

We are going to make this change in conjunction with adding a gitattributes file to our scaffold to allow autoconvertion for the file line endings as recommended here

Linked PRs: (387)[https://github.com/mobify/mobify-platform-sdks/pull/387]
Ticket: WEB-2316

Changes

  • remove line-break style rule

How To Test

  • (necessary config changes)
  • (necessary corresponding PRs)
  • (how to access the new / changed functionality -- fixtures, URLs)

Applicable Research Resources

  • (links, optional)

TODOs:

  • +1
  • Updated README
  • Updated CHANGELOG
  • (Other applicable TODOs)

@bendvc bendvc requested a review from olibrook May 23, 2019 21:01
Copy link

@olibrook olibrook left a comment

Choose a reason for hiding this comment

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

LGTM!

@bendvc bendvc merged commit 9973ed6 into develop May 29, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
3 participants