Skip to content

License: replace SPDX-Identifier misspelling in contract headers#558

Merged
sohkai merged 1 commit intoaragon:nextfrom
ottodevs:identitifer-fix
Oct 4, 2019
Merged

License: replace SPDX-Identifier misspelling in contract headers#558
sohkai merged 1 commit intoaragon:nextfrom
ottodevs:identitifer-fix

Conversation

@ottodevs
Copy link
Contributor

@ottodevs ottodevs commented Oct 2, 2019

Even though it was fun to spot projects reusing or forking aragon contracts: https://github.com/search?q=SPDX-License-Identitifer&type=Code

But I thought it might be interesting to fix the misspelling, in case any automated script might eventually parse that header to show the license.

Related: aragon/aragon-apps#1013

@welcome
Copy link

welcome bot commented Oct 2, 2019

Thanks for opening this pull request! Someone will review it soon 🔍

@coveralls
Copy link

Coverage Status

Coverage remained the same at 99.116% when pulling f07ae44 on ottodevs:identitifer-fix into 0fd1ff6 on aragon:next.

Copy link
Contributor

@sohkai sohkai left a comment

Choose a reason for hiding this comment

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

😅 That's one copy pasta gone unnoticed for far too long!

Good catch @ottodevs!

@sohkai sohkai changed the title Replace Identifier misspelling in license headers License: replace SPDX-Identifier misspelling in contract headers Oct 4, 2019
@sohkai sohkai merged commit 07d309f into aragon:next Oct 4, 2019
@welcome
Copy link

welcome bot commented Oct 4, 2019

Congrats on merging your first pull request! Aragon is proud of you 🦅
Eagle gif

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.

3 participants