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

Update ramda dependency #110

Merged
merged 1 commit into from
Feb 2, 2023
Merged

Update ramda dependency #110

merged 1 commit into from
Feb 2, 2023

Conversation

crash7
Copy link
Contributor

@crash7 crash7 commented Nov 10, 2022

I need to solve a couple of security issues that older ramda versions have. Check all the usage and it's safe to update to latest.

@icavina
Copy link

icavina commented Feb 2, 2023

@adepretis @fhelmberger @busypeoples @AutoSponge sorry for the inconvenience, not sure if you are still maintaining this repo, but it would be really nice if this can be merged. Thanks a lot in advance!

@adepretis adepretis merged commit 32bae41 into busypeoples:master Feb 2, 2023
@adepretis
Copy link
Collaborator

adepretis commented Feb 2, 2023

Just merged it. Unfortunately this project is no longer maintained @crash7 :-(

@busypeoples do you want to officially take over the repository? Otherwise would you be interested @crash7?

@icavina
Copy link

icavina commented Feb 2, 2023

Thanks @adepretis! Do you know if a new release will be triggered?

@adepretis
Copy link
Collaborator

@icavina uh, good point. As far as I remember we never had a pipeline in place and the package was published without organizational scope - so I guess @busypeoples has been maintaining this on npm. Can you publish it once again/once more?

@crash7
Copy link
Contributor Author

crash7 commented Feb 2, 2023

@adepretis thanks for merging this!

Happy to help with future PRs reviews or security updates!

@crash7
Copy link
Contributor Author

crash7 commented Feb 3, 2023

@adepretis I can also handle the npm publish if you want or if @busypeoples is not available.

@adepretis
Copy link
Collaborator

I don't have the credentials @crash7 ... trying to get a hold of @busypeoples currently, I'll get back to you ASAP

@busypeoples
Copy link
Owner

busypeoples commented Feb 3, 2023

@crash7 Thanks for the PR! Will publish an update. Thanks for merging @adepretis.

@busypeoples
Copy link
Owner

busypeoples commented Feb 3, 2023

Ok, published v.0.7.2 @crash7 @icavina @adepretis

@crash7
Copy link
Contributor Author

crash7 commented Feb 3, 2023

Thank you @busypeoples!

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.

4 participants