Skip to content
This repository has been archived by the owner on May 19, 2020. It is now read-only.

fix(README): grammar #195

Merged
merged 4 commits into from
Oct 28, 2019
Merged

fix(README): grammar #195

merged 4 commits into from
Oct 28, 2019

Conversation

gargarchit
Copy link
Contributor

Signed-off-by: Archit [email protected]

I found some Improper word choice as well as Improper formatting for that I have made changes and corrected that so, now It will be easier for everyone to read.
Please Merge this

Copy link
Member

@jolanglinais jolanglinais left a comment

Choose a reason for hiding this comment

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

You'll want to add a DCO (Developers Certificate of Originality) per the instructions:

Ensure you provide a DCO sign-off for your commits using the -s option of git commit. For more information see how this works.

Also you should create an issue in techdocs to track the same changes to all the repos so we have consistent README files.

Signed-off-by: Archit <[email protected]>
@gargarchit
Copy link
Contributor Author

@irmerk Thanks for the review, I have update changes and added DCO

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@jolanglinais
Copy link
Member

A couple changes, also make sure to rebase and push back up, then this can be merged.

Copy link
Member

@jolanglinais jolanglinais left a comment

Choose a reason for hiding this comment

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

Needs DCO again.

@jolanglinais jolanglinais changed the title Update README.md fix(README): grammar Oct 28, 2019
Signed-off-by: Archit <[email protected]>
@gargarchit
Copy link
Contributor Author

@irmerk Done and Thanks for this

@gargarchit
Copy link
Contributor Author

@irmerk I have Updated all changes

@jolanglinais jolanglinais merged commit 118d886 into accordproject:master Oct 28, 2019
@gargarchit gargarchit deleted the patch-1 branch October 28, 2019 15:36
@gargarchit
Copy link
Contributor Author

Thank You @irmerk

@jolanglinais
Copy link
Member

Email [email protected] so I can track your shipping info for Hacktoberfest.

@gargarchit
Copy link
Contributor Author

Thank You @irmerk I will email now

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants