-
Notifications
You must be signed in to change notification settings - Fork 749
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
Add comment in MessageType for readability #973
Conversation
Thank you for the improvement @devholic22 |
Thanks for your advice. But even if I keep running the commands you mentioned, I won't get a file that changes. Is there any other way? If there is, I will apply it right away. |
@tzolov The cause code is |
yeh, sorry. i've fixed the other error in c09286e Could you please rebase your branch, please use |
* Add comment in MessageType for readability * fix enum comment grammar
47a9874
to
a988020
Compare
Did I correct it well as you intended? I would appreciate it if you could check it :) and if you have time, please review another my Pull Request about Audio model. thanks! |
I still can see 2 commits? Did you squashed all commits? |
* Add comment in MessageType for readability * fix enum comment grammar Add comment in MessageType for readability
…/spring-ai into refactor-message-type
I squashed my commits using |
There are still 4 commits in this PR. I suggest you make a new PR with one commit that contains the changes you made the the file. |
I understand. you can check in new PR. |
@devholic22 can this PR be closed since the change has been delivered via #1000? |
resolved #972