-
Notifications
You must be signed in to change notification settings - Fork 8
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
Refactor using React & Typescript #1
Conversation
288d9ff
to
7684423
Compare
7684423
to
a3c009b
Compare
a3c009b
to
e1d1663
Compare
e1d1663
to
e0527c6
Compare
…Frontend] - Refactor store into multiple files - Cleanup console.logs
Create a workflow for linting pull requests
This feature will be added back using a different approach using jupyter-server-proxy.
- Also clean up unused dependencies and eslint config
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So... this is almost a complete rewrite of the extension (in ts)? Nice!
When you said rewrite in TS I though it was only the lab extension, but reusing components makes total sense.
I didn't have the time to check the ts files, only the others, so this is a half review. I decided to submit it since I'm on holidays and won't be able to finish until 1 week from now.
I like the organization of the different extensions.
Please gitignore the jar outputs, as they were commited and shouldn't. You also don't build the jars when building the extension. Are these decisions on purpose?
Side note: what are we going to do with nvm!swan_spark_conf
, which is the variable our users currently use?
- Update versioning to match `sparkmonitor` package on pypi - Updated readme comments - Remove makefile, commit hooks, tox, Dockerfile, and old requirements.txts - Remove jar file from git
Thanks for the review @diocas,
|
Summary of Changes
sparkmonitor
once merged