Skip to content

Conversation

@Bohreromir
Copy link
Contributor

It's my first time doing ts, so bare with me 😬

@alexlafroscia
Copy link
Owner

As far as the TypeScript usage goes, this all looks great!

@alexlafroscia alexlafroscia linked an issue Mar 16, 2021 that may be closed by this pull request
@alexlafroscia alexlafroscia changed the title #26 Add ability to process multiple partial folders Add ability to process multiple partial folders Mar 16, 2021
@alexlafroscia alexlafroscia changed the title #26 Add ability to process multiple partial folders Add ability to process multiple partial folders Mar 16, 2021
@Bohreromir
Copy link
Contributor Author

I didn't think about the fact that it would break the existing way. What do you think about my conversion?

Copy link
Owner

@alexlafroscia alexlafroscia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Functionality-wise, this looks great! Just a few little things I'd like to clean up before merging!

@Bohreromir
Copy link
Contributor Author

Thanks for the Feedback. I added a small hint to the README to let users know

Copy link
Owner

@alexlafroscia alexlafroscia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great! Thanks again for putting together this PR!

I’ll get a release cut with this feature today 👍

@alexlafroscia alexlafroscia merged commit fbcb39f into alexlafroscia:main Mar 18, 2021
@alexlafroscia
Copy link
Owner

Just released 1.2.0 that includes this fix!

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.

Support multiple partial directories

2 participants