Skip to content

Nis br method review #2204

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 9 commits into from
Aug 21, 2018

Conversation

cleitonmendonca
Copy link
Contributor

I've changed some reviews like orientation by @Arkni.

I've removed some spaces then I've chanced creation of date the new method nisBr.

Thank you!

Copy link
Member

@Arkni Arkni left a comment

Choose a reason for hiding this comment

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

LGTM :)

Thanks a lot for your contribution :) I'll proceed with merging in a moment.

@Arkni Arkni merged commit 01ce49c into jquery-validation:master Aug 21, 2018
@Arkni
Copy link
Member

Arkni commented Aug 21, 2018

@csmendonca,

By the way, you don't need to close a PR and create a new one with the new changes everytime a maintainer requested something. You can just do the changes and push your commit and they will be visible on the PR.

I recommend watching this ~38 minutes lessons from Kent C. Dodds, they are the best getting started on contributing to Open Source on GitHub: https://egghead.io/courses/how-to-contribute-to-an-open-source-project-on-github

And, thank you again for your contribution :)

@cleitonmendonca
Copy link
Contributor Author

Hi @Arkni,

Sorry about that, it's happing because that was my first contribution to Open Source on GitHub.
Thank you so much, I'll watch that video which you recommended.

@Arkni
Copy link
Member

Arkni commented Aug 21, 2018

Ah ok. I didn't pay attention.

Congratulation on your first contribution and Good luck :)

@cleitonmendonca
Copy link
Contributor Author

Hi @Arkni,

Thank you :)

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