Skip to content

Revert "fix: pin CodeBuild to Corretto 17 for AGP 7.4.2" - #126

Merged
cb-haripriyan merged 1 commit into
masterfrom
revert-125-fix/codebuild-corretto17
Aug 14, 2026
Merged

Revert "fix: pin CodeBuild to Corretto 17 for AGP 7.4.2"#126
cb-haripriyan merged 1 commit into
masterfrom
revert-125-fix/codebuild-corretto17

Conversation

@cb-jit-bhuin

@cb-jit-bhuin cb-jit-bhuin commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Reverts #125

Reverts the Corretto 17 configuration from the production buildspec. Removes the initial Java version check so the build uses the existing runtime setup.

@snyk-io

snyk-io Bot commented Aug 14, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues
Secrets 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@coderabbitai

coderabbitai Bot commented Aug 14, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Enterprise

Run ID: 69d7154a-402d-470a-99c9-8eb99e465610

📥 Commits

Reviewing files that changed from the base of the PR and between 1a3c862 and dc28477.

📒 Files selected for processing (1)
  • build-config/buildspec-prod.yml
💤 Files with no reviewable changes (1)
  • build-config/buildspec-prod.yml

Walkthrough

The production buildspec removes the Corretto 17 installation configuration and the initial Java version check. Existing build commands remain unchanged.

Changes

Production build configuration

Layer / File(s) Summary
Remove redundant Java setup
build-config/buildspec-prod.yml
The production buildspec removes the install phase runtime setup and the initial java -version command from the build phase.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to dc284

This PR reverts a localized production build configuration change; no actionable merge-blocking risk remains, so it is merge-ready after normal checks and review.

🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch

Comment @coderabbitai help to get the list of available commands.

@cb-haripriyan
cb-haripriyan merged commit 5853361 into master Aug 14, 2026
9 of 10 checks passed
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