Skip to content

Pycountry -> country_convertor - #77

Merged
irm-codebase merged 1 commit into
modelblocks-org:mainfrom
brynpickering:feat/pycountry-to-coco
Aug 3, 2026
Merged

irm-codebase merged 1 commit into
modelblocks-org:mainfrom
brynpickering:feat/pycountry-to-coco

Conversation

@brynpickering

@brynpickering brynpickering commented Aug 3, 2026 •

Copy link
Copy Markdown
Contributor

Fixes #58

Reviewer checklist

  • There are no pip dependencies in the module's environment files (workflow/envs/).
  • All rules use pathvars (e.g., <results>) in their inputs and outputs.
  • The integration test-suite is successful, including:
    • pre-commit.ci tests pass.
    • tests pass for all relevant OS configurations (linux, osx, windows).
  • Module documentation is up-to-date, including:
    • INTERFACE.yaml mentions all relevant pathvars and wildcards.
    • README.md describes how to use the module and has the necessary citations.

@irm-codebase
irm-codebase self-requested a review August 3, 2026 08:24

@irm-codebase irm-codebase left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you for the submission Bryn!
Looks good to me 🛩️

@irm-codebase

Copy link
Copy Markdown
Collaborator

@all-contributors please add @brynpickering for code

@allcontributors

Copy link
Copy Markdown
Contributor

@irm-codebase

I've put up a pull request to add @brynpickering! 🎉

@irm-codebase
irm-codebase merged commit 9e3f8fe into modelblocks-org:main Aug 3, 2026
4 checks passed
@brynpickering
brynpickering deleted the feat/pycountry-to-coco branch August 3, 2026 09:38
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.

use country_converter instead of pycountry

2 participants