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

Use standard SPDX identifier for license #7

Merged
merged 1 commit into from
Feb 4, 2021
Merged

Use standard SPDX identifier for license #7

merged 1 commit into from
Feb 4, 2021

Conversation

indeyets
Copy link
Contributor

@indeyets indeyets commented Feb 4, 2021

Various automatic tools do exact checks against strings listed here https://spdx.org/licenses/

cluster-key-slot uses upper-case name which isn't properly matched

Various automatic tools do exact checks against strings listed here https://spdx.org/licenses/

cluster-key-slot uses upper-case name which isn't properly matched
@CLAassistant
Copy link

CLAassistant commented Feb 4, 2021

CLA assistant check
All committers have signed the CLA.

@indeyets
Copy link
Contributor Author

indeyets commented Feb 4, 2021

@coveralls
Copy link

coveralls commented Feb 4, 2021

Pull Request Test Coverage Report for Build 33

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 31: 0.0%
Covered Lines: 50
Relevant Lines: 50

💛 - Coveralls

@Salakar Salakar merged commit b9e8fb2 into invertase:master Feb 4, 2021
@Salakar
Copy link
Member

Salakar commented Feb 4, 2021

Thank you, I will get this published tomorrow

@indeyets indeyets deleted the patch-1 branch February 5, 2021 10:10
@Uzlopak
Copy link

Uzlopak commented Aug 12, 2022

@Salakar

Can you please publish it?
Our pipeline did not handle the license identifier wll ;)

fastify/fastify-redis#158

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.

5 participants