Skip to content

Comments

Update copyright messages to xi-editor#246

Merged
raphlinus merged 2 commits intomasterfrom
authors
Aug 13, 2018
Merged

Update copyright messages to xi-editor#246
raphlinus merged 2 commits intomasterfrom
authors

Conversation

@raphlinus
Copy link
Member

Following guidance from https://opensource.google.com/docs/releasing/authors/
this patch changes the copyright message to "The xi-editor Authors" and
creates an AUTHORS file for contributors to record their name.

Also minor update to the contribution guidelines to remind people to add their
name to AUTHORS, and tweak the rules for approving and merging patches.

Following guidance from https://opensource.google.com/docs/releasing/authors/
this patch changes the copyright message to "The xi-editor Authors" and
creates an AUTHORS file for contributors to record their name.

Also minor update to the contribution guidelines to remind people to add their
name to AUTHORS, and tweak the rules for approving and merging patches.
@raphlinus
Copy link
Member Author

See xi-editor/xi-editor#778 for the parallel patch in the main xi-editor repo.


Please see [the xi-editor contribution guidelines](https://github.com/google/xi-editor/blob/master/CONTRIBUTING.md) about guidelines for contributors with write access.

If your name does not already appear in the [AUTHORS] file, please feel free to
Copy link
Member

Choose a reason for hiding this comment

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

Is this supposed to link to an included authors file, or to the author's file for xi-editor?

Copy link
Member Author

Choose a reason for hiding this comment

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

An included AUTHORS, I think I just forgot to git add it. I'll do it when I get to my workstation tomorrow.

@raphlinus raphlinus merged commit 105d7be into master Aug 13, 2018
@cmyr cmyr deleted the authors branch August 13, 2018 22:18
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.

2 participants