8000 Fix a typo in dictobject.c documentation by noahbkim · Pull Request #117515 · python/cpython · GitHub
[go: up one dir, main page]

Skip to content

Fix a typo i 8000 n dictobject.c documentation #117515

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

Merged
merged 1 commit into from
Apr 19, 2024
Merged

Conversation

noahbkim
Copy link
Contributor
@noahbkim noahbkim commented Apr 3, 2024

Fix a typo in dictobject.c documentation

This change fixes grammar in a comment in dictobject.c. Its true purpose is to kick off the CLA process so that I & other developers can contribute to CPython on behalf of Hudson River Trading.

Tagging @willingc as requested!

@ghost
Copy link
ghost commented Apr 3, 2024

All commit authors signed the Contributor License Agreement.
CLA signed

@bedevere-app
Copy link
bedevere-app bot commented Apr 3, 2024

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@noahbkim
Copy link
Contributor Author
noahbkim commented Apr 3, 2024

Seeing as the CLA bot points me back to the same form, could I get some clarification on

  1. Should I use my Github username for the Contributor field or my legal name?
  2. How do I distinguish between when I'm contributing on behalf of HRT and when I'm contributing individually?

@willingc willingc self-requested a review April 3, 2024 19:24
@willingc
Copy link
Contributor
willingc commented Apr 3, 2024

Seeing as the CLA bot points me back to the same form, could I get some clarification on

  1. Should I use my Github username for the Contributor field or my legal name?
  2. How do I distinguish between when I'm contributing on behalf of HRT and when I'm contributing individually?

Copying @ambv, our lead developer in residence, for additional guidance. Lukasz, Noah is a wonderful dev and part of the Hudson River Trading team. Could you please help to answer Noah's questions.

@noahbkim Do you see the following when you click on the cla button to sign?
Screenshot 2024-04-03 at 12 28 39 PM

If so, you should be able to authorize the CLA bot via your GitHub account. Or is that what is redirecting you to the form on the PSF website (https://www.python.org/psf/contrib/contrib-form/)?

Thanks Lukasz in advance for your help and Noah for your patience.

@noahbkim
Copy link
Contributor Author
noahbkim commented Apr 3, 2024

Do you see the following when you click on the cla button to sign?

I do, but the button seems to correspond to the individual CLA, not the organization one. I imagine the process is slightly different for when I contribute with my HRT hat on.

@tjstum
Copy link
tjstum commented Apr 4, 2024

Hey @willingc, @ambv and co.

I had sent this email to contributors@python.org a little over a year ago (and then bumped it to Sarah Swienckowski-Eckhart and Phyllis A. Dobbs a few times), but never actually got a response. If we can get some clarity on how the CLA process works for organizations (where people will need to be contributing under the HRT CLA when doing that as part of job duties, separate from individual duties), that would be a great help and let us move forward.

Hi there,

I had some questions about submitting the PSF Contributor Agreement for some contributions we'd like to propose to CPython.

The form has a field for "bugs.python.org username." My understanding is that Python has ceased using this site and uses GitHub Issues and Pull Requests instead (the header on that website says the same). How should I fill out that field?

Additionally, how do we indicate that a particular individual's GitHub account is associated with our entity's signed CLA? We have a GitHub organization, but individuals from our organization would be submitting Pull Requests using their own personal GitHub accounts. It wasn't totally clear to me how to go about this process.

I do see that the Docusign version no longer mentions bpo (though the PDF version still does). But there's still the question of how to manage associating GitHub accounts of HRT employees with this CLA. Should we have an authorized signatory Docusign the form with the account information (email address, I guess) for @noahbkim? And then repeat for each employee of HRT who needs to contribute under this CLA?

Thanks for helping us look into this!

@noahbkim
Copy link
Contributor Author

Bumping this thread!

@methane methane enabled auto-merge (squash) April 12, 2024 06:32
@ambv
Copy link
Contributor
ambv commented Apr 12, 2024

As for the corporate CLA, we need a corporate CLA form only once per organization. And we only need it in some situations. You, or another appropriate representative, need to submit this form on behalf of your employer (with their consent) whenever your work's copyright is in any way tied to your employment. You need to determine whether that's true for your jurisdiction and work situation.

The email on the corporate CLA form should ideally come from a domain belonging to the organization in question, and the personal identification of "Contributor" on the corporate CLA form should be the legal name of the company's representative and their signature, not a GitHub username. The form is now updated and no longer contains the outdated bugs.python.org field.

Now, after you do this, your first PR will still ask you to sign a personal CLA. You see, it's not enough for the PSF to accept a blanket form for "each and any" contributor falling under an organization as organization membership might not be well represented on GitHub at the time of contributing a PR, and so on.

In any case, you will be signing a personal CLA through GitHub no matter what. But we can't give you legal advice as to how you personally should distinguish personal from corporate contributions. For instance, in California, personal contribution performed on your own time using your personal hardware can still be challenged as work-related if it is related to your day job duties.

One popular choice to separate work from personal activity is to have your work email address listed on your GitHub account and to use that email for all git commits done as part of your work. This becomes tricky when some actions are performed through the GitHub UI which creates commits for you. So some people go as far as to have separate work accounts for GitHub.

@methane methane merged commit b624490 into python:main Apr 19, 2024
@noahbkim
Copy link
Contributor Author

Alright! We've signed the organization CLA on docusign as HRT Research LLC and I've just now signed the individual CLA. Is there anything else I need to do before I submit this other than amend the commit to use my HRT email address?

Thanks again for all the help 😄

@ambv
Copy link
Contributor
ambv commented Apr 20, 2024

That's it, Noah. Thank you!

@willingc
Copy link
Contributor

Thank you @ambv and glad @noahbkim that you were able to get stuff sorted out.

@noahbkim noahbkim deleted the cla-fix branch May 3, 2024 18:35
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.

5 participants
0