Skip to content

[CIVIS-11548] Install civis-r dependencies#9

Merged
d-weems merged 2 commits intomasterfrom
install-civis-r
Oct 2, 2025
Merged

[CIVIS-11548] Install civis-r dependencies#9
d-weems merged 2 commits intomasterfrom
install-civis-r

Conversation

@d-weems
Copy link
Copy Markdown

@d-weems d-weems commented Oct 2, 2025

Description

Required: Please provide a brief description of what this pull request is trying to accomplish.

Paired with this PR for civis-vscode. Installs libcurl and libxml, which are utilized when installing the civis-r package.

Context, Consequences, & Considerations

Required: Please step through the following list, pausing at each item to consider your change in relation to the item's context.
Check the box to mark that it applies, and enter your relevant notes under the item.

  • Security: This has security implications. This includes (but not limited to) adding users, modifying user/app permissions, network rules/policies, changing a system interconnection, or changing an authorization strategy.
    • This PR does not require security review. These changes are part of a project plan that has already undergone security review. The link is provided below.
    • This PR requires security review. Add the security label to this PR then request a review from the Security Code Reviewers Team.
  • Execution: This change requires commands to be run outside of the normal merge.
  • Impact: This change may cause service interruptions.
  • Testing: How did you test this change (unit tests, acceptance tests, etc.)? Did you do any manual testing?
  • Testing: How will you confirm this change once it's merged?

Confirm civis-vscode builds and installs civis-r package

  • Documentation: Documentation to reflect this change has been added to Confluence or Zendesk.
  • All items of the checklist have been considered and this PR description is complete.

@d-weems d-weems requested review from a team and RobertoCarrilloAvila and removed request for a team October 2, 2025 17:25
Copy link
Copy Markdown
Collaborator

@thatguyinabeanie thatguyinabeanie left a comment

Choose a reason for hiding this comment

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

LggTM

@d-weems d-weems merged commit 6e10ef5 into master Oct 2, 2025
@civis-robot
Copy link
Copy Markdown

Pull Request Merged!!! This build is running now.

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.

3 participants