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

Change Elastic-2.0 license to plain text format #2637

Merged
merged 2 commits into from
Dec 20, 2024
Merged

Conversation

zhongjiajie
Copy link
Contributor

Seems current Elastic-2.0 license in the markdown format, we should convert it into plain text format, because others licenses are in the plain text format

Seems current Elastic-2.0 license in the markdown format, we
should convert it into plain text format, because others licenses
are in the plain text format
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

@xsuchy xsuchy added this to the 3.26.0 milestone Dec 20, 2024
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.

The tests fails.
The change needs to be done in test/simpleTestForGenerator/Elastic-2.0.txt too.

@zhongjiajie
Copy link
Contributor Author

Hi @xsuchy, I also change the license in test directory, could you please approval the workflow CI run?

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

@xsuchy
Copy link
Collaborator

xsuchy commented Dec 20, 2024

Thank you.

@xsuchy xsuchy merged commit cb00a23 into spdx:main Dec 20, 2024
2 checks passed
@zhongjiajie zhongjiajie deleted the patch-1 branch December 21, 2024 01:25
@swinslow swinslow added the XML markup change potential change or addition to XML markup in license label Dec 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
XML markup change potential change or addition to XML markup in license
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants