Skip to content

L3xi42 patch 2 #820

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

Closed
wants to merge 3 commits into from
Closed

L3xi42 patch 2 #820

wants to merge 3 commits into from

Conversation

l3xi42
Copy link

@l3xi42 l3xi42 commented Oct 8, 2022

I corrected some grammar mistakes

Copy link
Collaborator

@saragibby saragibby left a comment

Choose a reason for hiding this comment

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

Thank you for spotting these items and getting them fixed!

@saragibby
Copy link
Collaborator

@l3xi42 these changes look good to merge. All we need is your agreement to the contributor license agreement and the conflicts resolved. Once those are complete, we can merge. Thank you!

@saragibby saragibby added documentation Improvements or additions to documentation changes requested Requested changes to pull request labels Oct 27, 2022
@saragibby saragibby self-assigned this Oct 27, 2022
@saragibby
Copy link
Collaborator

@l3xi42 along with needing your contributor agreement, there is now some conflicts to be resolved on this PR. Once both are taking care of we can get this merged. Thanks!

@saragibby
Copy link
Collaborator

Closing this pull request as it is missing contributor agreement and has unresolved conflicts.

@saragibby saragibby closed this Mar 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes requested Requested changes to pull request documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants