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

Decaffeinate, convert to (relatively) modern JS #620

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

Conversation

szakharchenko
Copy link

No description provided.

We do not intend to support this. Stick to using new please.

Reverts: 05c7f43
decaffeinate --use-js-modules *.coffee

decaffeinate v6.1.8 used.
This makes installation from e.g. Git URLs possible.
decaffeinate *.coffee

decaffeinate v6.1.8 used.
…what

Use Object.assign for option overrides. Avoid using || {} when it's
clear that the LHS is an object. Use arrow functions everywhere where
'this' is unused. Use Array.map for xpath node comprehension.
This also drops the documentation generation script.
@a2937
Copy link

a2937 commented Sep 22, 2024

You never actually finished this pull request. The tests are referencing files that don't exist.

@szakharchenko
Copy link
Author

I'm afraid I have migrated from xml2js+xmlbuilder to xmlbuilder2 shortly after submitting this pull request (I know other alternatives are suggested in neighboring issues); given the current maintenance status, I have little motivation to do any updates, add any potentially missing files, resolve conflicts, etc. Sorry.

@a2937
Copy link

a2937 commented Oct 24, 2024

I was trying to build a fork that's somewhat maintained since I unfortunately found this library. I'd love all the help I can get.

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.

2 participants