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

Remove dependency on read-pkg #2863

Merged
merged 1 commit into from Oct 17, 2021
Merged

Conversation

alessandroasm
Copy link
Contributor

@alessandroasm alessandroasm commented Oct 5, 2021

As discussed on Issue #2853, this package was only used in one place and just to read a single property on a package.json file.

As discussed on Issue avajs#2863, this package was only used in one place and just to
read a single property on a package.json file.
@alessandroasm
Copy link
Contributor Author

@live627 Please let me know if I need to change something :)

@live627
Copy link
Contributor

live627 commented Oct 8, 2021 via email

@alessandroasm alessandroasm marked this pull request as ready for review October 8, 2021 10:54
@alessandroasm
Copy link
Contributor Author

@live627 No worries! :)

The PR is now ready for review.

@novemberborn novemberborn linked an issue Oct 17, 2021 that may be closed by this pull request
@novemberborn novemberborn changed the title Remove read-pkg package from dependencies Remove dependency on read-pkg Oct 17, 2021
@novemberborn novemberborn merged commit 7727e61 into avajs:main Oct 17, 2021
@novemberborn
Copy link
Member

Thanks @alessandroasm!

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.

readPackageSync() could be simplified, 17 fewer deps
3 participants