-
Notifications
You must be signed in to change notification settings - Fork 482
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] Add new osci author kenny zhang #2047
[OSCI] Add new osci author kenny zhang #2047
Conversation
Signed-off-by: Jinlin Zhang <[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.
@kennyzhang0819 can you move the file to _authors/osci_2023
? Sorry there was a mistake in my instructions. Apologies for that.
@kennyzhang0819 you are missing the signoff on that latest commit. Can you run this to resolve:
|
Signed-off-by: Jinlin Zhang <[email protected]>
Signed-off-by: Kris Freedain <[email protected]> Signed-off-by: Jinlin Zhang <[email protected]>
Signed-off-by: Kris Freedain <[email protected]> Signed-off-by: Jinlin Zhang <[email protected]>
Signed-off-by: Jinlin Zhang <[email protected]>
11dc292
to
52532bb
Compare
Hi, can you check again and see if this is correct? Also, I realized there are two pull requests, should I close the other one? |
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.
LGTM
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 great!~
Description
I added new osci author kenny zhang (myself)
Issues Resolved
[List any issues this PR will resolve]
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the BSD-3-Clause License.