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

Create history-OSI-SPDX (2nd try) #2504

Merged
merged 4 commits into from
Aug 6, 2024
Merged

Create history-OSI-SPDX (2nd try) #2504

merged 4 commits into from
Aug 6, 2024

Conversation

jlovejoy
Copy link
Member

@jlovejoy jlovejoy commented Jul 2, 2024

made this as new PR b/c #1738 was failing tests. See #1738 for background if need be.

Signed-off-by: Jilayne Lovejoy

@jlovejoy jlovejoy self-assigned this Jul 2, 2024
@jlovejoy jlovejoy added this to the 3.25.0 milestone Jul 2, 2024
@jlovejoy jlovejoy requested a review from swinslow July 2, 2024 04:41
@xsuchy
Copy link
Collaborator

xsuchy commented Jul 2, 2024

The file does not have .md extension therefore is GitHub will not format it as Markdow. Can you please add the suffix to the filename, please?

@xsuchy
Copy link
Collaborator

xsuchy commented Jul 2, 2024

Otherwise technically LGTM.

@jlovejoy
Copy link
Member Author

jlovejoy commented Jul 9, 2024

The file does not have .md extension therefore is GitHub will not format it as Markdow. Can you please add the suffix to the filename, please?

where is the facepalm emoji when you need it?!? :)

Copy link
Collaborator

@xsuchy xsuchy left a comment

Choose a reason for hiding this comment

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

+1

Copy link
Member

@swinslow swinslow left a comment

Choose a reason for hiding this comment

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

Hi @jlovejoy, I've gone through and added a few specific questions / edits to consider below.

For the overall history part at the beginning, I think this makes a lot of sense to capture as a historical record for development of the early license list and interactions with OSI.

For the particular issues described further down, I'm assuming that these are meant more as a temporary gathering-together of related issues? In the long run, I'm guessing that these would ideally end up either addressed by OSI where needed, perhaps via issues in https://github.com/OpenSourceOrg/licenses (I know you've captured some of these below). Or, perhaps more appropriate to include comments in the "Notes" fields for applicable license IDs on the SPDX License List, since otherwise I'm guessing ordinary users won't know to look here for them. Either way, as an initial place to store these details, I think this works.

DOCS/history-OSI-SPDX.md Outdated Show resolved Hide resolved
DOCS/history-OSI-SPDX.md Outdated Show resolved Hide resolved
DOCS/history-OSI-SPDX.md Outdated Show resolved Hide resolved
DOCS/history-OSI-SPDX.md Outdated Show resolved Hide resolved
DOCS/history-OSI-SPDX.md Outdated Show resolved Hide resolved
@jlovejoy
Copy link
Member Author

jlovejoy commented Aug 6, 2024

@swinslow - addressed your suggestions, want to have a quick look and then merge?

Copy link

@Thompson1985 Thompson1985 left a comment

Choose a reason for hiding this comment

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

Thank you

@Thompson1985
Copy link

Thank you

Copy link
Member

@swinslow swinslow left a comment

Choose a reason for hiding this comment

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

LGTM, thank you @jlovejoy!

@swinslow swinslow merged commit 9a2bc4e into spdx:main Aug 6, 2024
2 checks passed
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.

None yet

4 participants