-
Notifications
You must be signed in to change notification settings - Fork 137
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
Fixes errors in doc header and operator install command #506
Conversation
Codecov Report
@@ Coverage Diff @@
## master #506 +/- ##
============================================
+ Coverage 58.83% 59.64% +0.80%
- Complexity 1707 1753 +46
============================================
Files 206 205 -1
Lines 11508 11488 -20
Branches 1030 1028 -2
============================================
+ Hits 6771 6852 +81
+ Misses 4323 4232 -91
+ Partials 414 404 -10
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
docs/uniffle-migration-guide.md
Outdated
|
||
http://www.apache.org/licenses/LICENSE-2.0 | ||
|
||
http://www.apache.org/licenses/LICENSE-2.0 |
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.
Why do we need to change this? Looks like other docs also uses this format.
https://github.com/apache/incubator-uniffle/blob/master/docs/server_guide.md?plain=1
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.
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.
Can we just move the lisense into comments instead of yaml header?
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.
Comments? I don't get your point. I have uploaded latest commit and fixed its style.
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, thanks @zuston for the fix.
What changes were proposed in this pull request?
Fixes errors in doc header and operator install command
Why are the changes needed?
Fixes errors in doc header and operator install command
Does this PR introduce any user-facing change?
No
How was this patch tested?
Don't need