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

PR: Update PR template to link to developer COI and clarify other text #7928

Merged
merged 2 commits into from
Sep 21, 2018

Conversation

CAM-Gerlach
Copy link
Member

Pull Request Checklist

  • Read and followed this repo's Contributing Guidelines
  • Based your PR on the latest version of the correct branch (master or 3.x)
  • Ensured your pull request hasn't eliminated unrelated blank lines/spaces,
    modified the spyder/defaults directory, or added new icons/assets
  • Described your changes and the motivation for them below

Affirmation

By submitting this Pull Request, I affirm that:

  • I release the content of this PR under Spyder's MIT (Expat) license.
  • I hold the copyright to all the content in this PR, have not copied any of it
    directly or indirectly from any source other than the public domain,
    and no other party owns the rights to this work.
I affirm all of the above: CAM-Gerlach

Description of Changes

Updates the pull request template to simply link to the standard and accepted Developer COI used by many open source projects, rather than lengthier bespoke legal text included previously. Also does some minor cleanup, refinements and clarification to the template text.

Copy link
Member

@ccordoba12 ccordoba12 left a comment

Choose a reason for hiding this comment

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

Thanks!

@ccordoba12 ccordoba12 merged commit b17df0a into spyder-ide:3.x Sep 21, 2018
ccordoba12 added a commit that referenced this pull request Sep 21, 2018
@CAM-Gerlach CAM-Gerlach deleted the pr-template-devcoi branch February 12, 2019 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants