Skip to content

[#2258] Update Netlify build #15

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
Merged
sopa301 merged 1 commit into from
Apr 10, 2025
Merged

[#2258] Update Netlify build #15

sopa301 merged 1 commit into from
Apr 10, 2025

Conversation

@CYX22222003
Copy link
Member

@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
Member 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
@sopa301 sopa301 force-pushed the update-netlify-build branch from 7c011a7 to 2d978bb Compare March 18, 2025 08:13
@sopa301 sopa301 removed the request for review from gok99 March 24, 2025 09:34
@sopa301 sopa301 merged commit 22a1e34 into reposense:master Apr 10, 2025
2 checks passed
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

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