Skip to content
This repository has been archived by the owner on Jul 20, 2020. It is now read-only.

Homework, zero errors with Eslint #2

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

erikmaide
Copy link

@erikmaide erikmaide commented Feb 9, 2020

Fixed errors with import & require in 1place

example.js Outdated
@@ -1,53 +1,50 @@
const Lammas = require('./lammas');
import Sheep from './sheep';
Copy link
Owner

Choose a reason for hiding this comment

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

this is fine, just adding comment that eslint failing here...

require and import in same project is not the best, will explain more in next lesson when we do more deep drive into linting...

@eritikass eritikass added the done label Mar 27, 2020
@eritikass eritikass added the CI label Mar 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants