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

non-babel version #14

Closed
rynop opened this issue May 24, 2017 · 2 comments
Closed

non-babel version #14

rynop opened this issue May 24, 2017 · 2 comments
Assignees

Comments

@rynop
Copy link

rynop commented May 24, 2017

Do you have a non-babel version of this module? Node v7.10 has all the es2017 features I need, so I don't need to add the babel dependencies.

Or is the best/only way to clone and run your build then include your code that is put into the dist dir?

@pokusew pokusew self-assigned this May 30, 2017
pokusew added a commit that referenced this issue Jun 2, 2017
Improve examples (#16, #7)
Add new keywords into package.json
Add example from readme into examples folder
@pokusew
Copy link
Owner

pokusew commented Jun 2, 2017

Hi @rynop,

this library uses for example import/export (ES6 modules) and these are not supported in any Node.js version yet. Otherwise all needed ES2015 features, that nfc-pcsc lib uses, are supported in 7.10, but to support all 7.x Node.js versions, I need to still transpile some things.

But you don't need to use Babel in your app at all. When you install nfc-pcsc via npm/yarn and then require it in your code, the transpiled source from dist is used, so you don't need to worry about anything.

As a part of the recent improvements (issues fixes), I have changed the way how to import/require nfc-pcsc, removed unnecessary Babel presets and plugins, dropped support for older Node.js versions and hugely improved examples and README.

These changes are not published yet, but I will release them as a part of the upcoming 0.6.0 release in a week or two.

Definitely take a look at these FAQs, which I have added to README to better explain the situation:

Hope it helps.

PS Don't forget to star ⭐️ my library, if you find it useful. 😃 Thanks.

@rynop
Copy link
Author

rynop commented Jun 7, 2017

Thanks @pokusew I really appreciate the thorough response and readme update. I look forward to v 0.6.0. Closing this now, because as you mention it does not require babel to use.

@rynop rynop closed this as completed Jun 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants