Skip to content

closes #647 by fixing a logging inconsistency and updating versioning#672

Merged
saramonzon merged 3 commits into
BU-ISCIII:developfrom
saramonzon:saramonzon/issue647
Jun 18, 2025
Merged

closes #647 by fixing a logging inconsistency and updating versioning#672
saramonzon merged 3 commits into
BU-ISCIII:developfrom
saramonzon:saramonzon/issue647

Conversation

@saramonzon

@saramonzon saramonzon commented Jun 17, 2025

Copy link
Copy Markdown
Member

Summary

This PR closes #647 by improving error handling and execution flow in the metadata module.

Changes included:

  • Fixed an issue where no message was printed to stdout/stderr when an exception was raised, making debugging harder.
  • Fixed logic to ensure that example entries are not silently skipped when empty — now they are properly validated or reported.

PR Checklist

  • This comment contains a description of changes (with reason).
  • Make sure your code lints (black and flake8).
  • CHANGELOG.md is updated.
  • README.md is updated (including new tool citations and authors/contributors).

@jaimeozaez jaimeozaez left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@saramonzon saramonzon merged commit 46bc1b1 into BU-ISCIII:develop Jun 18, 2025
17 checks passed
@saramonzon saramonzon deleted the saramonzon/issue647 branch June 25, 2025 14:12
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.

2 participants