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.

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

Update the license headers for Nextcloud 19 #20241

Merged
merged 1 commit into from Apr 1, 2020
Merged

Update the license headers for Nextcloud 19 #20241

merged 1 commit into from Apr 1, 2020

Conversation

ChristophWurst
Copy link
Member

First I thought seeing my name in so many files was a bug, but it appears to be because of #20168

@ChristophWurst ChristophWurst added the 3. to review Waiting for reviews label Mar 31, 2020
@ChristophWurst ChristophWurst added this to the Nextcloud 19 milestone Mar 31, 2020
@ChristophWurst ChristophWurst self-assigned this Mar 31, 2020
@ChristophWurst ChristophWurst added this to TO REVIEW (max 4 PRs) in Christoph's Tasks via automation Mar 31, 2020
@tcitworld
Copy link
Member

tcitworld commented Mar 31, 2020 •
edited

nextcloud@tcit.fr isn't a typo. :)
I'll like all the tcit@tcit.fr changed to nextcloud@tcit.fr, since you're doing this.

EDIT : Oh, okay, you're using information from git, my bad.

Christoph's Tasks automation moved this from TO REVIEW (max 4 PRs) to TO INTEGRATE Mar 31, 2020
@kesselb
Copy link
Contributor

kesselb commented Mar 31, 2020

nextcloud@tcit.fr isn't a typo. :)
I'll like all the tcit@tcit.fr changed to nextcloud@tcit.fr, since you're doing this.

Good news! Just add the mapping to https://github.com/nextcloud/server/blob/master/.mailmap

tcitworld added a commit that referenced this pull request Mar 31, 2020
@tcitworld
Add myself to .mailmap
6ae77a8 
See #20241
tcitworld added a commit that referenced this pull request Mar 31, 2020
@tcitworld
Add myself to .mailmap
e9a0117 
See #20241
@tcitworld tcitworld mentioned this pull request Mar 31, 2020
Merged
@tcitworld
Copy link
Member

Good news! Just add the mapping to /.mailmap@master

Many thanks, forgot this existed at all!

@ChristophWurst ChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Mar 31, 2020
tcitworld
tcitworld approved these changes Mar 31, 2020
View reviewed changes
Copy link
Member

@tcitworld tcitworld left a comment

Choose a reason for hiding this comment

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

🙏

@ChristophWurst
Update the license headers for Nextcloud 19
1a9330c 
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@rullzer rullzer merged commit 19ca921 into master Apr 1, 2020
Christoph's Tasks automation moved this from TO INTEGRATE to DONE Apr 1, 2020
@rullzer rullzer deleted the fix/license-headers-19 branch April 1, 2020 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rullzer rullzer rullzer approved these changes

@tcitworld tcitworld tcitworld approved these changes

@marcoambrosini marcoambrosini marcoambrosini approved these changes

@gary-kim gary-kim gary-kim approved these changes

@bergart bergart Awaiting requested review from bergart

@GretaD GretaD Awaiting requested review from GretaD

Assignees

@ChristophWurst ChristophWurst

Labels
4. to release Ready to be released and/or waiting for tests to finish
Projects
No open projects
Christoph's Tasks
DONE
Milestone
Nextcloud 19
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@ChristophWurst @tcitworld @kesselb @rullzer @marcoambrosini @gary-kim