Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add UUID for all DDF files #7742

Merged
merged 1 commit into from
May 13, 2024
Merged

Add UUID for all DDF files #7742

merged 1 commit into from
May 13, 2024

Conversation

Zehir
Copy link
Collaborator

@Zehir Zehir commented May 5, 2024

The UUID serves the following purposes:

  • It provides a unique identifier for a DDF, which remains consistent even if the file is relocated.
  • It facilitates the grouping of bundles that contain different versions of the same file.
  • Each DDF should have an unique UUID. For the new DDF an new UUID will be generated when the PR get merged.
  • They are only used by the DDF bundle store.

The UUID has been appended to all existing DDFs in the repository.

The pull request was created using the following command:

for dir in $(find . -type d);
  do   ddf-tools bulk uuid --store-url {URL} --store-token {TOKEN} $dir;
done

The UUID was inserted immediately after the schema. Maintaining the formatting and using the same newline character, either \r\n or \n

For new DDFs that are currently in a pull request, they will be updated by the new validation action at a later stage.

@Zehir Zehir reopened this May 13, 2024
@manup manup merged commit c820b7d into dresden-elektronik:master May 13, 2024
2 checks passed
@Zehir Zehir deleted the add-uuid branch May 13, 2024 13:01
@bcutter
Copy link

bcutter commented Aug 18, 2024

What happens to custom DDFs without a set UUID? No benefits as listed in the PR description or any side effects?

@bcutter
Copy link

bcutter commented Sep 1, 2024

What happens to custom DDFs without a set UUID? No benefits as listed in the PR description or any side effects?

@Zehir

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants