Skip to content

Conversation

@dmmcinty
Copy link

@dmmcinty dmmcinty commented Apr 9, 2018

No description provided.

@dmmcinty dmmcinty requested a review from spruce-bruce April 9, 2018 22:46
Copy link
Contributor

@spruce-bruce spruce-bruce left a comment

Choose a reason for hiding this comment

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

Just a question

src/DropNCrop.js Outdated
className="dropzone-validation"
>
{value && value.error}
{value && value.src ? (
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you explain these changes? Did you run prettier on this or something?

When I'm reviewing a PR having large blocks of formatting changes to unrelated code makes it harder for me to understand 100% of what's going on.

Copy link
Author

Choose a reason for hiding this comment

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

prettier ran automatically, so i'd assumed it was intended that way. let me recommit but without prettier

@dmmcinty
Copy link
Author

should be good now

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