Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR updates the metadata handling system with comprehensive style standardization and new functionality. The changes focus on converting string literals to double quotes, updating schema URLs, and adding version management capabilities.
- String literal standardization from single quotes to double quotes across all Python files
- Schema URL updates to reference the official Quadriga schema repository
- New version management script (
update_version_from_tag.py) for automated version updates
Reviewed Changes
Copilot reviewed 13 out of 13 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| quadriga/metadata/utils.py | Quote standardization and schema URL update |
| quadriga/metadata/update_version_from_tag.py | New script for automated version updates from git tags |
| quadriga/metadata/update_citation_cff.py | Quote standardization and version handling improvements |
| quadriga/metadata/run_all.py | Quote standardization |
| quadriga/metadata/extract_from_book_config.py | Quote standardization and schema URL update |
| quadriga/metadata/create_bibtex.py | Quote standardization and import formatting |
| quadriga/metadata/init.py | Added new module import |
| metadata.yml | Schema URL update and restructured metadata |
| CITATION.cff | Updated metadata values and version information |
| CITATION.bib | Updated citation information |
| .python-version | New Python version specification |
| .github/workflows/update-metadata.yml | New workflow for automated metadata updates |
| .github/workflows/deploy-book.yml | Updated to use Python version file |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| "book-version" | ||
| ] | ||
| updates_made = True | ||
| logging.info(f"Updated version to: {metadata["book-version"]}") |
There was a problem hiding this comment.
This line contains a syntax error due to improperly nested quotes. The f-string contains double quotes which conflict with the outer double quotes. Use single quotes for the dictionary key or escape the inner quotes.
|
sorry, didn't realize you did this. i now did it again (I didn't think …) |
No description provided.