Skip to content

Conversation

jhnwllr
Copy link
Collaborator

@jhnwllr jhnwllr commented Jun 26, 2025

@jhnwllr jhnwllr requested a review from Copilot June 26, 2025 14:24
Copilot

This comment was marked as outdated.

@jhnwllr jhnwllr requested a review from Copilot July 1, 2025 10:50
Copilot

This comment was marked as outdated.

@jhnwllr jhnwllr requested a review from Copilot September 3, 2025 06:40
Copy link

@Copilot 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 support for installation-related functionality to the rgbif package, allowing users to search for installations and retrieve installation metadata using installation UUIDs. The implementation follows the existing package patterns for similar GBIF API endpoints.

  • Adds installation search functionality with filtering options
  • Implements installation metadata retrieval functions for various endpoints
  • Includes comprehensive test coverage with VCR cassettes for API responses

Reviewed Changes

Copilot reviewed 10 out of 14 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
R/installation_search.R New function for searching installations with various filter parameters
R/intallation_uuid_funs.R New functions for retrieving installation metadata by UUID
tests/testthat/test-installation_search.R Test suite for installation search functionality
tests/testthat/test-installation_uuid_funs.R Test suite for installation UUID functions
tests/fixtures/*.yml VCR cassettes containing recorded API responses for tests
man/*.Rd Documentation files for the new functions
NAMESPACE Export declarations for new functions
DESCRIPTION Version bump

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

jhnwllr and others added 7 commits September 3, 2025 08:46
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@jhnwllr jhnwllr merged commit a59dd84 into master Sep 3, 2025
7 checks passed
@jhnwllr jhnwllr deleted the installations branch September 3, 2025 07:23
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.

1 participant