Skip to content

Conversation

maxhbr
Copy link
Member

@maxhbr maxhbr commented Sep 18, 2023

All URLs are valid download locations.

This closes #761 and #762.

fix spdx#761 fix spdx#762 Signed-off-by: Maximilian Huber <maximilian.huber@tngtech.com>
@maxhbr maxhbr force-pushed the maxhbr/fix761and762 branch from 275d245 to 0b8f6dc Compare September 18, 2023 09:38
Copy link
Collaborator

@armintaenzertng armintaenzertng left a comment

Choose a reason for hiding this comment

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

Please add the test cases from the issues

@maxhbr
Copy link
Member Author

maxhbr commented Sep 19, 2023

I would prefer to not extend the current URL testcases, to avoid confusing URL semantics with recommendations on which URLs to use.

Copy link
Collaborator

@armintaenzertng armintaenzertng left a comment

Choose a reason for hiding this comment

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

OK, then it looks good to me :)

@armintaenzertng armintaenzertng merged commit ed9a135 into spdx:main Sep 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants