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.

Sign up for GitHub

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

Jump to bottom

Remove arguments and code related to the html-report #1310

Closed
5 tasks done
amahussein opened this issue Aug 22, 2024 · 0 comments · Fixed by #1311
Closed
5 tasks done

Remove arguments and code related to the html-report #1310

amahussein opened this issue Aug 22, 2024 · 0 comments · Fixed by #1311
Assignees
@amahussein
Labels
build core_tools Scope the core module (scala)

Comments

@amahussein
Copy link
Collaborator

amahussein commented Aug 22, 2024 •
edited
Loading

Is your feature request related to a problem? Please describe.

This issue is to track removing the HTML report from the code. This will reduce the size of the tools jar since it gets rid of all the JS sources.

CC: FYI @mattahrens @viadea @kuhushukla
@amahussein amahussein added documentation Improvements or additions to documentation core_tools Scope the core module (scala) build labels Aug 22, 2024
@amahussein amahussein self-assigned this Aug 22, 2024
@amahussein amahussein removed the documentation Improvements or additions to documentation label Aug 22, 2024
This was referenced Aug 22, 2024
Closed
Closed
Closed
@amahussein amahussein changed the title [FEA] Remove arguments and code related to the html-report Remove arguments and code related to the html-report Aug 22, 2024
amahussein added a commit to amahussein/spark-rapids-tools that referenced this issue Aug 22, 2024
@amahussein
Remove arguments and code related to the html-report
10cbe79 
Signed-off-by: Ahmed Hussein <[email protected]>

Fixes NVIDIA#1310

- Remove argument html-report from Core tools
- Remove mvn task that copies the UI resources into the jar resources
- Remove copyrights related to the JS sources
- Remove the scala code that generate the html report
@amahussein amahussein mentioned this issue Aug 22, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@amahussein amahussein

Labels
build core_tools Scope the core module (scala)
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove arguments and code related to the html-report amahussein/spark-rapids-tools
1 participant
@amahussein