Skip to content

Update SPDX license Ids & exceptions to 3.20 #9028

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Jun 20, 2023
Merged

Update SPDX license Ids & exceptions to 3.20 #9028

merged 4 commits into from
Jun 20, 2023

Conversation

Kleidukos
Copy link
Member

@Kleidukos Kleidukos commented Jun 17, 2023 •
edited
Loading

Please include the following checklist in your PR:

Bonus points for added automated tests!

@Kleidukos Kleidukos mentioned this pull request Jun 17, 2023
Closed
2 tasks
ffaf1
ffaf1 approved these changes Jun 17, 2023
View reviewed changes
Copy link
Collaborator

@ffaf1 ffaf1 left a comment

Choose a reason for hiding this comment

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

Very good

@Kleidukos Kleidukos added squash+merge me Tell Mergify Bot to squash-merge attention: needs-review labels Jun 17, 2023
ulysses4ever
ulysses4ever approved these changes Jun 17, 2023
View reviewed changes
Copy link
Collaborator

@ulysses4ever ulysses4ever left a comment

Choose a reason for hiding this comment

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

Updating these licenses is not beautiful. I hope at some point we can think of something that wouldn't require to touch so many files...

@ffaf1
Copy link
Collaborator

ffaf1 commented Jun 17, 2023

I also wonder if we need all of those (SchemeReport?) and if there is a way to import OSI/FSF seal of approval too.

@mergify mergify bot added the merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days label Jun 19, 2023
@ulysses4ever
Copy link
Collaborator

@mergify refresh

@mergify
Copy link
Contributor

mergify bot commented Jun 19, 2023

refresh

✅ Pull request refreshed

@ulysses4ever
Copy link
Collaborator

ulysses4ever commented Jun 19, 2023 •
edited
Loading

I'm clueless as to why Mergify doesn't act on this one. Althought it's been "only" one hour since the "merge delay passed" label was applied.

@Kleidukos
Copy link
Member Author

Look at me, I am the merge bot now

@Kleidukos Kleidukos merged commit c97e487 into haskell:3.10 Jun 20, 2023
@Kleidukos Kleidukos deleted the 3.10.2.0-update-spdx-licences branch June 20, 2023 08:59
@Mikolaj Mikolaj mentioned this pull request Aug 25, 2023
Open
9 tasks
@fgaz fgaz mentioned this pull request Sep 14, 2023
Closed
Kleidukos added a commit that referenced this pull request Sep 14, 2023
@Kleidukos
Revert "Update SPDX license Ids & exceptions to 3.20 (#9028)"
a2a6df3 
This reverts commit c97e487.
@Kleidukos Kleidukos mentioned this pull request Sep 14, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ulysses4ever ulysses4ever ulysses4ever approved these changes

@ffaf1 ffaf1 ffaf1 approved these changes

Assignees
No one assigned
Labels
merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days squash+merge me Tell Mergify Bot to squash-merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Kleidukos @ffaf1 @ulysses4ever