-
Notifications
You must be signed in to change notification settings - Fork 471
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
Is this repo still being maintained? #274
Comments
No comment until now. @benjamine would be good to have a statement from you. |
I'm not currently adding new features, but yes will review any fixes sent. |
@benjamine is there a changelog? It's fine if you don't have time, I understand. |
Hi @Offirmo thank you!, I don't have a changelog, but should. |
@benjamine I would recommend recruiting an additional trusted maintainer or two. It's been 2 years since the last commit and even longer since an actual meaningful commit was made. I just ran into an issue where the version being used by the jsondiffpatch dependency Now I'm using https://www.npmjs.com/package/jsondiffpatch-rc which doesn't feel great. It would be great if at the bare minimum the dependencies for this repo could be updated. Thanks! PS I would be happy to take on a maintainer role for this repo as we are still using it in our tools |
hi folks, apologies for the lack of replies.
I agree at least some help keeping dependencies up to date and any patches
would be great!
I’d love peering with a volunteer on this.
…On Mon, Jun 5, 2023 at 4:19 PM Thomas Rich ***@***.***> wrote:
@benjamine <https://github.com/benjamine> I would recommend recruiting an
additional trusted maintainer or two. It's been 2 years since the last
commit and even longer since an actual meaningful commit was made.
I just ran into an issue where the version being used by the jsondiffpatch
dependency chalk is causing a build issue with newer build tools (like
vite) - #315 <#315>
Now I'm using https://www.npmjs.com/package/jsondiffpatch-rc which
doesn't feel great.
It would be great if at the bare minimum the dependencies for this repo
could be updated.
Thanks!
PS I would be happy to take on a maintainer role for this repo as we are
still using it in our tools
—
Reply to this email directly, view it on GitHub
<#274 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAC54KLFLWZXWL3BLLC6UQLXJYWMPANCNFSM4I7REKBQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Hey @benjamine, I'm the maintainer of redux-devtools which depends on this project and I would be interested in helping maintain this package if the offer still stands. Thanks! |
@Methuselah96 that'd be awesome! of course, will get in touch to get it done 💪 , thank you! |
Looks like there hasn't been any PR merged in quite a while.. ?
@benjamine what's your status?
The text was updated successfully, but these errors were encountered: