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

[#2258] Update Netlify build #15

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

[#2258] Update Netlify build #15

wants to merge 3 commits into from

Conversation

CYX22222003
Copy link

@CYX22222003 CYX22222003 commented Jan 31, 2025 •
edited by sopa301
Loading

Fix reposense/RepoSense#2258

Proposed commit message

Update build and deploy script for Netlify

The current instructions to deploy to Netlify no longer work as the
default build image only has Java 8.

Let's add code to manually set up Java 11 on Netlify for deployment.

Other information

Attempt to manually set up Java 11 environment on Netlify deployment server.
This is the test website and its build and deployment logs

@CYX22222003 CYX22222003 changed the title Update Netlify build [#2258] Update Netlify build Feb 1, 2025
@CYX22222003 CYX22222003 requested review from a team, gok99 and sopa301 and removed request for a team and gok99 February 1, 2025 04:53
@CYX22222003 CYX22222003 requested a review from gok99 February 9, 2025 21:53
@CYX22222003 CYX22222003 mentioned this pull request Feb 11, 2025
Closed
@CYX22222003
Copy link
Author

CYX22222003 commented Feb 14, 2025 •
edited
Loading

We will update ug in PR instead.

@CYX22222003 CYX22222003 reopened this Feb 14, 2025
sopa301
sopa301 approved these changes Feb 14, 2025
View reviewed changes
Copy link
Contributor

@sopa301 sopa301 left a comment

Choose a reason for hiding this comment

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

LGTM

@CYX22222003 CYX22222003 removed the request for review from a team February 14, 2025 03:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sopa301 sopa301 sopa301 approved these changes

@gok99 gok99 Awaiting requested review from gok99

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Netlify Deployment Instructions No Longer Work
2 participants
@CYX22222003 @sopa301