Skip to content

HACKING.rst: add clarifying note to LP CLA process section#789

Merged
OddBloke merged 2 commits into
canonical:masterfrom
OddBloke:doc
Jan 26, 2021
Merged

HACKING.rst: add clarifying note to LP CLA process section#789
OddBloke merged 2 commits into
canonical:masterfrom
OddBloke:doc

Conversation

@OddBloke
Copy link
Copy Markdown
Collaborator

Proposed Commit Message

HACKING.rst: add clarifying note to LP CLA process section

We've had some first time contributors still end up trying to submit via
Launchpad: this commit underlines that this is not necessary.

Test Steps

Build the docs and observe:

cla-note

Checklist:

  • My code follows the process laid out in the documentation
  • I have updated or added any unit tests accordingly
  • I have updated or added any documentation accordingly

We've had some first time contributors still end up trying to submit via
Launchpad: this commit underlines that this is not necessary.
Copy link
Copy Markdown
Contributor

@mitechie mitechie left a comment

Choose a reason for hiding this comment

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

After going through the process I think that changing the title of this section to set the condition scope would be useful.

e.g.

Transferring CLA Signatures from Launchpad to Github

to perhaps

If you previously signed the CLA through Launchpad...

Which helps set the optional/condition of this section upfront.

The note in this PR is +1 either way.

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