Skip to content

Add missing UUIDs to YAML file entries lacking them#889

Merged
haesleinhuepf merged 8 commits into
mainfrom
git-bob-mod-77gIJ7lr9f
Sep 16, 2025
Merged

Add missing UUIDs to YAML file entries lacking them#889
haesleinhuepf merged 8 commits into
mainfrom
git-bob-mod-77gIJ7lr9f

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

This message was generated by git-bob (version: 0.27.2, model: github_models:gpt-4o, log), an experimental AI-based assistant. It can make mistakes and has limitations. Check its messages carefully.

I implemented a Python script, add_uuid_to_resources.py, to address issue #888. This script scans the resources/nfdi4bioimage.yml file for entries under the resources section that lack a uuid field. It generates and assigns UUID version 4 values to these entries. The script also interacts with the repository by creating a new branch, committing the updated YAML file, and sending a pull request using utility functions provided in scripts/_github_utilities.py.

closes #888

…L file and create a pull request via `_github_utilities`.
@haesleinhuepf
Copy link
Copy Markdown
Member

git-bob implement a main thingy that calls the function if I execute the script from the terminal.

@github-actions
Copy link
Copy Markdown
Contributor Author

This message was generated by git-bob (version: 0.27.2, model: github_models:gpt-4o, log), an experimental AI-based assistant. It can make mistakes and has limitations. Check its messages carefully.

To address issue #888, a Python script add_uuid_to_resources.py was created to identify and add missing UUIDs to entries in the resources/nfdi4bioimage.yml file under the resources section. The script generates UUID version 4 values for these entries and automates the process of creating a branch, committing changes, and submitting a pull request using existing GitHub utility functions. In response to feedback, a __main__ block was added to add_uuid_to_resources.py, allowing the script to be executed directly from the terminal.

@cthoyt
Copy link
Copy Markdown

cthoyt commented Sep 15, 2025

hi @haesleinhuepf thanks for looking into this. I think having the UUIDs in your upstream YAML is the perfect solution

@haesleinhuepf haesleinhuepf merged commit 860ed98 into main Sep 16, 2025
2 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.

Add UUIDs to yml

2 participants