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

Pva/rdfbrowser 540 npm evs explore build process #335

Merged
merged 17 commits into from
Feb 13, 2025

Conversation

peter-va
Copy link
Collaborator

@peter-va peter-va requested a review from bcarlsenca February 12, 2025 19:22
Copy link
Collaborator

@bcarlsenca bcarlsenca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lots of changes, but mostly file renaming. There are two more things:

  1. Work on the top level README so that it properly explains how to build and run the project. Right now we have "Building the Java Webapp" and "Launching EVS-EXPLORE for development" sections (which are probably wrong). We should also have a "Making a tag for deployment" where we talk about the staged process - build the fronend, build the web, commit changes then make a tag.
  2. Make sure to test that "make run" still works from the makefile.

Copy link
Collaborator

@bcarlsenca bcarlsenca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@peter-va peter-va merged commit e5437a3 into develop Feb 13, 2025
2 checks passed
@peter-va peter-va deleted the pva/RDFBROWSER-540-npm-evs-explore-build-process branch February 13, 2025 21:41
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.

2 participants