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

Contribution points 1 #831

Merged
merged 9 commits into from
Oct 30, 2023
Merged

Conversation

TheoMcCabe
Copy link
Collaborator

@TheoMcCabe TheoMcCabe commented Oct 30, 2023

So i made a bunch of contributions as thoemccabe@mac.com which is an incorrect email and doesnt link to my github account... and so they dont count towards my contributions!

... this my vein PR to rebase those changes to my github user and recommit them (as opposed to rewriting git history on main which would be risky)

@sweep-ai
Copy link
Contributor

sweep-ai bot commented Oct 30, 2023

Apply Sweep Rules to your PR?

  • Apply: Ensure all new functions and classes have very clear, concise and up-to-date docstrings. Take gpt_engineer/ai.py as a good example.

@TheoMcCabe TheoMcCabe closed this Oct 30, 2023
@TheoMcCabe TheoMcCabe reopened this Oct 30, 2023
@TheoMcCabe TheoMcCabe marked this pull request as ready for review October 30, 2023 12:01
@TheoMcCabe TheoMcCabe marked this pull request as draft October 30, 2023 15:56
@ATheorell ATheorell marked this pull request as ready for review October 30, 2023 16:07
@ATheorell ATheorell merged commit 4725df5 into AntonOsika:main Oct 30, 2023
9 checks passed
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