Skip to content

Conversation

@sky76093016
Copy link
Contributor

@sky76093016 sky76093016 commented Nov 6, 2020

What changes were proposed in this pull request?

I corrected the spelling of the word.

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-4434

How was this patch tested?

No need.

Copy link
Member

@ayushtkn ayushtkn left a comment

Choose a reason for hiding this comment

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

LGTM

@ayushtkn
Copy link
Member

ayushtkn commented Nov 7, 2020

Can you assign the jira to your name, or can you help me with your jira id, If you aren't able to do so. Would be great if you can put up a comment in the jira itself, I will pick the id from there

@sky76093016
Copy link
Contributor Author

@ayushtkn I have already replied in JIRA, thank you.

Copy link
Contributor

@cku328 cku328 left a comment

Choose a reason for hiding this comment

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

LGTM+1. Thanks @sky76093016 for working on this.
Let's wait for full green build.

@ayushtkn ayushtkn merged commit 0ad6aba into apache:master Nov 8, 2020
@ayushtkn
Copy link
Member

ayushtkn commented Nov 8, 2020

Merged PR, Thanx @sky76093016 for the contribution and @cku328 for the review!!!

@cku328
Copy link
Contributor

cku328 commented Nov 10, 2020

Thanks @ayushtkn for merging it.

Should we modify Git commit message to make it look the same as the others?
image
A blank character is missing...

@ayushtkn
Copy link
Member

ayushtkn commented Nov 10, 2020

That is ok. You can't change the commit message without reverting. And this commit will still stay.
Until the jira id is not wrong, things would be good

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.

3 participants