Skip to content

Conversation

@ignatiusreza
Copy link
Contributor

@ignatiusreza ignatiusreza commented May 20, 2021

Initial checklist

  • I read the support docs
  • I read the contributing guide
  • I agree to follow the code of conduct
  • I searched issues and couldn’t find anything (or linked relevant results below)
  • If applicable, I’ve added docs and tests

Description of changes

re-export all and one from index.js, closes #53

originally I thought we could also re-export revert, and wrap, but looking at the implementation, it looks like those 2 are quite specialized for processing only some type of node, so I refrain from re-exporting them..

these helper functions are useful when writting custom handlers
@wooorm wooorm changed the title re-export all and one from index.js Add exports of one, all helpers May 20, 2021
@wooorm wooorm merged commit 77ef216 into syntax-tree:main May 20, 2021
@wooorm wooorm added 💪 phase/solved Post is done 🗄 area/interface This affects the public interface 🦋 type/enhancement This is great to have 🧒 semver/minor This is backwards-compatible change labels May 20, 2021
@wooorm
Copy link
Member

wooorm commented May 20, 2021

Released, thanks @ignatiusreza!

@ignatiusreza ignatiusreza deleted the export-helpers branch May 20, 2021 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🗄 area/interface This affects the public interface 💪 phase/solved Post is done 🧒 semver/minor This is backwards-compatible change 🦋 type/enhancement This is great to have

Development

Successfully merging this pull request may close these issues.

all, one, etc should be re-exported in main index.js?

2 participants