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

[issue-381] Add tag-value writer #396

Merged

Conversation

armintaenzertng
Copy link
Collaborator

This takes over from #395.
Fixes #381.

This PR includes code from #379, so depending on which PR is merged first, there will probably be merge conflicts with the one or the other when rebasing.

Signed-off-by: Meret Behrens <meret.behrens@tngtech.com>
Signed-off-by: Meret Behrens <meret.behrens@tngtech.com>
Signed-off-by: Meret Behrens <meret.behrens@tngtech.com>
@armintaenzertng armintaenzertng changed the title [issue-381] Add tag value writer [issue-381] Add tag-value writer Dec 30, 2022
Signed-off-by: Armin Tänzer <armin.taenzer@tngtech.com>
Signed-off-by: Armin Tänzer <armin.taenzer@tngtech.com>
We do not want to take invalid files into account when writing. For such cases, validation should be performed before the writing process.

Signed-off-by: Armin Tänzer <armin.taenzer@tngtech.com>
also rename write_package_verification_code, erase a line in the test example and an erroneous break

Signed-off-by: Armin Tänzer <armin.taenzer@tngtech.com>
Signed-off-by: Armin Tänzer <armin.taenzer@tngtech.com>
@armintaenzertng armintaenzertng force-pushed the addTagValueWriter branch 2 times, most recently from e7d0f68 to e0d0dbd Compare January 2, 2023 10:42
Signed-off-by: Armin Tänzer <armin.taenzer@tngtech.com>
@armintaenzertng armintaenzertng merged commit fee4094 into spdx:refactor-python-tools Jan 2, 2023
@armintaenzertng armintaenzertng deleted the addTagValueWriter branch January 2, 2023 10:54
Jwya

This comment was marked as spam.

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.

None yet

3 participants