Skip to content
This repository has been archived by the owner on May 31, 2024. It is now read-only.

eslint and babel update WIP #2704

Open
wants to merge 2 commits into
base: devel
Choose a base branch
from
Open

Conversation

Neobii
Copy link
Contributor

@Neobii Neobii commented Mar 23, 2021

I'm working on updating the eslint and babel and some issues are still there.

I've added the correct plugins so maybe there are just some linting issues throwing errors. github issue

 3:7  error  Parsing error: /home/adaml/Desktop/Vulcan/packages/vulcan-redux/lib/server/main.js: Support for the experimental syntax 'exportDefaultFrom' isn't currently enabled (3:8):

  1 | import './reduxInitialState';
  2 |
> 3 | export setupRedux from './setupRedux';
    |        ^
  4 |
  5 | export * from '../modules/index';

@Neobii
Copy link
Contributor Author

Neobii commented Mar 23, 2021

I notice that most of vulcan's packages are using package.js... but some meteor-mocha and vulcan-backoffice have a package.json. I can switch them to package.js or make a linting rule to ignore the json files. (Meteor mocha has both a json and a js file)

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

Successfully merging this pull request may close these issues.

1 participant