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

Upgrade ESLint #5

Open
wants to merge 8 commits into
base: AP-6-create-onboarding-app
Choose a base branch
from
Open

Conversation

sohkai
Copy link
Contributor

@sohkai sohkai commented May 20, 2016

Upgrades the ESLint configuration with mostly cosmetic changes to fix new errors. The BlocksGroup component was moved out of BlocksConveyor since it's gotten large enough to deserve it's own component.

@sohkai sohkai force-pushed the upgrade-eslint branch 2 times, most recently from 9c1ae44 to cef09e2 Compare May 26, 2016 08:58
@@ -26,9 +27,10 @@ const UploaderEnhancements = [
AscribeBlobUploader,
AscribeRequestKeyUploader
];
const Uploader = UploaderEnhancements.reduce((Uploader, Enhancer) => Enhancer(Uploader), ReactS3FineUploader);
const EnhancedUploader = UploaderEnhancements.reduce((Uploader, Enhancer) => Enhancer(Uploader),
ReactS3FineUploader);
Copy link

Choose a reason for hiding this comment

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

This looks weird. Maybe don't use an inline arrow function but one that wraps with {} and has a return value.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, agreed; do you think it'd look better on a single line or with the return (either causes a linting error that we'd turn off)?

Copy link
Contributor Author

@sohkai sohkai Jun 14, 2016

Choose a reason for hiding this comment

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

Decided to go with:

const EnhancedUploader = UploaderEnhancements.reduce(
    (Uploader, Enhancer) => Enhancer(Uploader),
    ReactS3FineUploader
);

I think that looks better?

@TimDaub
Copy link

TimDaub commented Jun 13, 2016

Apart from the suggestions 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants