NRLMMD-GEOIPS / NRLMMD-GEOIPS/pluginify

Migrate PluginValidationError from GeoIPS into Pluginify

Open
#16 0 comments 0 reactions 1 assignee View on GitHub

@coleman-m is already working on this.

Since May 22, 2026.

  • #17 by @coleman-m — open
Dominant language
Python
Stars
1
Forks
0
PR merge metrics
No merged PRs in 30d

Description

Requested Update

Description
  • Move the PluginValidationError from GeoIPS into Pluginify
Background and Motivation
  • Geoips PR 1269 introduces the PluginValidationError, a new exception class in errors.py that wraps the raw pydantic ValidationError and replaces it with a concise summary grouped by field. The error identifies the plugin by name, interface, package, and file path, then lists only the unique error messages per field rather than repeating them across every union variant.
Alternative Solutions
  • Keep that logic within GeoIPS
Environment
Code to demonstrate issue
Checklist for Completion
  • Replicate the PR contents in the Pluginify repository
  • Close the original GeoIPS PR, pointing towards the new PR
  • Ensure that the error reporting still works

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.