Skip to content

Conversation

Jatidude
Copy link
Contributor

This is my first ever contribution to an open source repository so if i did something wrong just let me know. I changed the README to reflect the changes that I made but It could probable be cleaned up.

@Jatidude Jatidude changed the title Creat option to maintain structure of your data. Create option to maintain structure of your data. Aug 27, 2019
Copy link
Owner

@philipstanislaus philipstanislaus left a comment

Choose a reason for hiding this comment

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

Thanks, that's great! I've left a couple of comments.

@Jatidude
Copy link
Contributor Author

I made a commit does that automatically get applied? I'm not fully show how this works

@philipstanislaus
Copy link
Owner

@Jatidude thanks, looks great!

I did some cleanup, please check. If you approve, I'll merge.

Thanks for your first FOSS contribution!

Copy link
Contributor Author

@Jatidude Jatidude left a comment

Choose a reason for hiding this comment

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

@philipstanislaus Looks perfectly fine to me thanks!

@Jatidude
Copy link
Contributor Author

@philipstanislaus is there something else I need to do before this gets merged? Sorry if I'm jumping the gun just want to make sure there's nothing more you need me to do.

@philipstanislaus philipstanislaus changed the base branch from master to dataOption September 3, 2019 15:33
@philipstanislaus philipstanislaus merged commit bc38bcb into philipstanislaus:dataOption Sep 3, 2019
@philipstanislaus
Copy link
Owner

@Jatidude sorry for the late reply, I was travelling. Just merged, will deploy in a bit!

philipstanislaus added a commit that referenced this pull request Sep 3, 2019
Add option to maintain structure of your data. (#6)
@philipstanislaus
Copy link
Owner

@Jatidude all done, released as v1.2.0: https://www.npmjs.com/package/performant-array-to-tree

Thanks for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants