Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@PietrH
Copy link
Member

@PietrH PietrH commented Oct 7, 2025

No description provided.

@PietrH PietrH marked this pull request as ready for review October 7, 2025 12:19
@PietrH PietrH requested a review from Copilot October 7, 2025 12:19
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 prepares release v0.5.1 which addresses a locale-dependent sorting issue by replacing R's base sort() function with stringr::str_sort() for consistent ordering across different computer locales.

Key Changes

  • Replaced sort() with stringr::str_sort() in four list functions to ensure consistent ordering regardless of locale
  • Updated version number from 0.5.0 to 0.5.1 in DESCRIPTION file
  • Added changelog entry documenting the locale-independent sorting improvement

Reviewed Changes

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

Show a summary per file
File Description
R/list_scientific_names.R Replaced sort() with stringr::str_sort() for consistent scientific name ordering
R/list_cpod_project_codes.R Replaced sort() with stringr::str_sort() for consistent CPOD project code ordering
R/list_animal_project_codes.R Replaced sort() with stringr::str_sort() for consistent animal project code ordering
R/list_acoustic_project_codes.R Replaced sort() with stringr::str_sort() for consistent acoustic project code ordering
NEWS.md Added changelog entry for version 0.5.1 describing the locale-independent sorting fix
DESCRIPTION Updated package version from 0.5.0 to 0.5.1

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

@PietrH PietrH linked an issue Oct 7, 2025 that may be closed by this pull request
19 tasks
@PietrH PietrH merged commit 9b9318c into main Oct 7, 2025
6 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.

Release etnservice v0.5.1

2 participants