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

Monorepo #412

Closed
20 tasks done
zalmoxisus opened this issue Nov 17, 2018 · 15 comments
Closed
20 tasks done

Monorepo #412

zalmoxisus opened this issue Nov 17, 2018 · 15 comments

Comments

@zalmoxisus
Copy link
Collaborator

zalmoxisus commented Nov 17, 2018

As discussed before, moving all redux-devtools related repositories to reduxjs organization doesn't look sustainable (though it's still open for discussion). A better way, especially for new contributors, would be to have everything in a monorepo. We could do that in redux-devtools-extension as discussed in zalmoxisus/redux-devtools-extension#412 since there's more activity, but using just redux-devtools name sounds better and everything started here. So the plan is to keep only this repository, the rest will be archived and the code moved here. I'll add credits for every readme. Maybe we could even add authors of the monitors in the extension, just need an UI for that (something like a monitors store).

The list of repos to be merged (some will be renamed as suggested bellow):

These repositories could still be maintained outside as either having a maintainer or no changes needed in future (?):

If everybody agrees on, we still have to finish the work in progress in opened PRs, so first will need to:

  • fix all issues, except feature requests, for 2.16,
  • finish stack trace functionality started in Show stack traces zalmoxisus/remotedev-app#43 and move to a separate redux-devtools-trace-monitor package,
  • finish all refactoring, work in progress, implement new UI and all the updates for 3.0.
@alexkuz
Copy link
Collaborator

alexkuz commented Nov 18, 2018

I'm okay with moving it wherever :)

@zalmoxisus
Copy link
Collaborator Author

zalmoxisus commented Nov 18, 2018

@alexkuz thanks for the reply! I granted permissions for this repository to you. Would you like to continue to maintain react-json-tree or better to move it here too? There are no critical issues with it, just few improvements we'll need merged.

@alexkuz
Copy link
Collaborator

alexkuz commented Nov 21, 2018

I'd better move it too since I barely support my projects lately

@zalmoxisus
Copy link
Collaborator Author

Thanks @alexkuz! When you have some time, please add me as a collaborator to that repo and on npm.

@alexkuz
Copy link
Collaborator

alexkuz commented Nov 25, 2018

@zalmoxisus done

@herodrigues
Copy link

herodrigues commented Dec 9, 2018

That's a clever idea, because there are a lot of "redux-ish" development applications.
It should be something way more simple just like react-devtools.

A browser extension and a standalone Electron app would be enough.
Themes and monitors should be part of the main package.

Also, there are too many projects for simple things and they all have many dependencies (most of them are outdated).
i'm willing to help on this 😄

@zalmoxisus
Copy link
Collaborator Author

@herodrigues definitely, it'll be easier to maintain and contribute. There are use cases where using monitors separately is useful (at least till we offer the ability to run the extension directly from client part solving serialization perf issues). Also react-native-debugger and integrations for non-redux are relying on our packages. The monorepo lets us maintain everything like a single package, but still providing the same multiple npm packages.

I did first steps in #416, which seems to work seamlessly. Will move more packages next week.

If you can help with configurations, upgrading dependences, tests, docs..., it'll be much appreciated ☺️

@zalmoxisus
Copy link
Collaborator Author

And yes, providing an electron app for redux-devtools-app which will silently launch websocket server (without adding remotedev-server to the client app) is needed. I was exploring using Chrome Native Messaging instead, but it still requires a messaging host, so there are no advantages comparing to current approach of using socketcluster/ remotedev-server bridge.

@herodrigues
Copy link

@zalmoxisus how is it going with the browser extension?

@Methuselah96
Copy link
Member

@alexkuz Do you mind if we also move react-base16-styling over here? react-json-tree relies on it pretty heavily and it would be convenient to be able to update its dependencies and/or types.

@alexkuz
Copy link
Collaborator

alexkuz commented Jul 15, 2020

@Methuselah96 sorry, I've missed the mention - I don't mind, tell me if I should add you or @zalmoxisus as collaborator.

@Methuselah96
Copy link
Member

Methuselah96 commented Jul 15, 2020

@alexkuz No worries. thanks. If you could add me that would great. @zalmoxisus isn't active right now.

@Methuselah96
Copy link
Member

Methuselah96 commented Aug 15, 2020

@alexkuz Can we move react-dock here as well? It has some React 16 deprecation warnings that it would be great to resolve (see alexkuz/react-dock#40 (comment)). npm username is methuselah96. Thanks!

@alexkuz
Copy link
Collaborator

alexkuz commented Aug 15, 2020

@Methuselah96 sure, you should have your access now.

@Methuselah96
Copy link
Member

Closing since all of the packages mentioned have been moved here. See #530 for future plans.

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

No branches or pull requests

4 participants