-
Notifications
You must be signed in to change notification settings - Fork 483
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
[OSCI] Adds new author VJoshi380 #2207
Conversation
Signed-off-by: Vedang Joshi <[email protected]>
Signed-off-by: Vedang Joshi <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple small suggestions. Otherwise looks good.
github: VJoshi380 | ||
--- | ||
|
||
**Vedang Joshi** I love security through obsucrity :O |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure if the spelling is the joke, but if not:
**Vedang Joshi** I love security through obsucrity :O | |
**Vedang Joshi** I love security through obscurity :O |
short_name: | ||
twitter: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove these if not needed
short_name: | |
twitter: |
@VJoshi380 once you address Josh's comments we are good to go, thank you! |
Signed-off-by: Vedang Joshi <[email protected]>
Thanks for letting me know about the issues. I've removed the unused sections and fixed the typo. Let me know if any other fixes need to be made. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
Description
Placing Vedang Joshi in the authors section of the website repository.
By submitting this pull request, I confirm that my contribution is made under the terms of the BSD-3-Clause License.