Skip to content
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

fixed simple typo in run_fine_tuning.py #128

Merged
merged 1 commit into from
Aug 14, 2023

Conversation

olson-ibm
Copy link
Contributor

fixed simple typo in run_fine_tuning.py

Signed-off-by: Joe Olson <joe.olson@ibm.com>
@olson-ibm
Copy link
Contributor Author

olson-ibm commented Aug 14, 2023

Sorry for the confusion here. I'm still hazy on the proper procedure for signing commits. Do I have to have my ssh public key added to this repository?

@rawkintrevo
Copy link
Contributor

Nice work @olson-ibm , @gkumbhat / @alex-jw-brooks this issue is blocking me- could we get an expedited review? please and thank you.

@alex-jw-brooks
Copy link
Collaborator

alex-jw-brooks commented Aug 14, 2023

LGTM, thanks for the fix!

I saw that you fixed the sign off, thanks @olson-ibm! IIRC signoff is tied to GPG keys. You shouldn't need to do anything specific for this repo, but you made need to get it set up for your account (which it seems like you may have already!)

If you end up needing to go back and sign multiple commits in the future, you can do it with a git rebase HEAD~n --signoff to sign the last n commits & force push

@alex-jw-brooks alex-jw-brooks merged commit 9739ecd into caikit:main Aug 14, 2023
4 checks passed
@olson-ibm olson-ibm deleted the bugfix_signed_jjo branch August 21, 2023 15:22
Ssukriti pushed a commit to Ssukriti/caikit-nlp that referenced this pull request Aug 24, 2023
Fix typo in run fine tuning

Signed-off-by: Sukriti-Sharma4 <sukriti.sharma4@ibm.com>
gkumbhat pushed a commit to gkumbhat/caikit-nlp that referenced this pull request Aug 24, 2023
Fix typo in run fine tuning
Signed-off-by: gkumbhat <kumbhat.gaurav@gmail.com>
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.

None yet

3 participants