Skip to content

add force argument to java unpack and install#86

Merged
e-kotov merged 2 commits intomainfrom
56-add-a-force-argument-to-java_install
Sep 22, 2025
Merged

add force argument to java unpack and install#86
e-kotov merged 2 commits intomainfrom
56-add-a-force-argument-to-java_install

Conversation

@e-kotov
Copy link
Owner

@e-kotov e-kotov commented Sep 22, 2025

No description provided.

@e-kotov e-kotov linked an issue Sep 22, 2025 that may be closed by this pull request
@e-kotov e-kotov requested a review from Copilot September 22, 2025 11:34
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a force parameter to the java_unpack and java_install functions to enable overwriting existing Java installations.

  • Adds force parameter with default value FALSE to both functions
  • Implements force removal logic in java_unpack to delete existing installations when force=TRUE
  • Updates documentation and function signatures to include the new parameter

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
R/java_unpack.R Adds force parameter and implements forced removal of existing installations
R/java_install.R Adds force parameter and passes it through to java_unpack
man/java_unpack.Rd Updates documentation to include force parameter
man/java_install.Rd Updates documentation to include force parameter

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@e-kotov e-kotov merged commit 55dc10a into main Sep 22, 2025
6 of 7 checks passed
@e-kotov e-kotov deleted the 56-add-a-force-argument-to-java_install branch September 22, 2025 11:59
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.

Add a force argument to java_install()

1 participant