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

Support node12 and goodies #27

Merged

Conversation

AdrieanKhisbe
Copy link
Contributor

This Pr a set of enhancements that started with tackling issue of node 12 support (
Internal of REPLServer were reworked, breaking repl.history dependency: [DEP0124] DeprecationWarning: REPLServer.rli is deprecated),

and escalated quickly into a lot of different features and refactors:

  • replace repl.history to support node12
  • rework internal to facilitate creation of new instances. (motivated for tests reasons)
  • introduce new tests and improve coverage
  • introduce yargs to help with flag parsing (and configuration via env vars)
  • introduce .lodash repl command (see doc introduced in README)
  • customizable prompt

=)

disclamer, I made repl-story as replacement for repl.history that does now support new repl APIs
@borisdiakur
Copy link
Owner

@AdrieanKhisbe Thank you for the contribution! That’s a lot indeed. Will take some time until I have reviewed it all, but I’ll try to make some time soon.

@borisdiakur
Copy link
Owner

@AdrieanKhisbe Looks good. 👍Except for the table in README.md 😬Can you have a look?

@AdrieanKhisbe
Copy link
Contributor Author

Hello @borisdiakur :)

That was quick!

Indeed, I should have had a look 😁 and not make it blindly 🙈

Seem like I had some old orgmode syntax habit 😁
@AdrieanKhisbe
Copy link
Contributor Author

et voilà, I'll put that blunder on old orgmode habits.
That should be fixed now :)

Capture d’écran 2020-07-06 à 09 55 06

@borisdiakur borisdiakur merged commit 739bcfc into borisdiakur:master Jul 16, 2020
@borisdiakur
Copy link
Owner

Thanks again @AdrieanKhisbe 🙏
Just published version 3.0.0 🚀

@AdrieanKhisbe AdrieanKhisbe deleted the support-node12-and-goodies branch July 17, 2020 09:50
@AdrieanKhisbe
Copy link
Contributor Author

Awesome @borisdiakur ! =)
Thanks to you too!!

(Might come back one of these days if ever I resume my prototype about introducing lodash/fp signature doc from within the REPL 📖... 🤞)

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.

None yet

2 participants