Skip to content
This repository has been archived by the owner on Mar 8, 2019. It is now read-only.

Update Readme.srt #157

Merged
merged 1 commit into from
Oct 25, 2018
Merged

Update Readme.srt #157

merged 1 commit into from
Oct 25, 2018

Conversation

coder579
Copy link
Contributor

Fixed some typos.

@coder579
Update Readme.srt
cc6d614 
Fixed some typos.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 92.127% when pulling cc6d614 on coder579:master into fa7516e on mozilla:master.

@coder579
Copy link
Contributor Author

So my PR won't be accepted?

@davehunt davehunt merged commit ff76509 into mozilla:master Oct 25, 2018
@davehunt
Copy link
Member

Accepted, approved, merged, and appreciated! Thank you @coder579!

@coder579
Copy link
Contributor Author

coder579 commented Oct 26, 2018 •
edited
Loading

Thank you so much! @davehunt
You paved my way into the world of Open Source. It was my first ever Open Source contribution. Although it was quite small, but it was necessary to get me started.

@stephendonner
Copy link
Contributor

Welcome, @coder579; I still can remember my first Open Source contribution back in 1998, which not only gave me a job, set me off on an amazing career; cheers!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@davehunt davehunt davehunt approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@coder579 @coveralls @davehunt @stephendonner