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

[MINOR] docs: Fix the license format of remote_merge_guide.md #2314

Merged
merged 1 commit into from
Dec 30, 2024

Conversation

xianjingfeng
Copy link
Member

What changes were proposed in this pull request?

Fix the license format of remote_merge_guide.md

Why are the changes needed?

If we open https://github.com/apache/incubator-uniffle/blob/master/docs/remote_merge_guide.md, we can see the following warning.

Error in user YAML: (<unknown>): could not find expected ':' while scanning a simple key at line 6 column 1

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Unnecessary

Copy link

Test Results

 2 966 files  ±0   2 966 suites  ±0   6h 33m 6s ⏱️ + 4m 10s
 1 097 tests ±0   1 095 ✅ ±0   2 💤 ±0  0 ❌ ±0 
13 750 runs  ±0  13 720 ✅ ±0  30 💤 ±0  0 ❌ ±0 

Results for commit e377df7. ± Comparison against base commit e9e5904.

@xianjingfeng xianjingfeng merged commit 580b4f2 into apache:master Dec 30, 2024
41 of 43 checks passed
@xianjingfeng
Copy link
Member Author

@zhengchenyu Thanks for your review.

@xianjingfeng xianjingfeng deleted the minor branch December 30, 2024 14:03
jerqi pushed a commit that referenced this pull request Jan 1, 2025
### What changes were proposed in this pull request?
Fix the license format of remote_merge_guide.md

### Why are the changes needed?
If we open https://github.com/apache/incubator-uniffle/blob/master/docs/remote_merge_guide.md, we can see the following warning.

Error in user YAML: (<unknown>): could not find expected ':' while scanning a simple key at line 6 column 1

### Does this PR introduce any user-facing change?
No.

### How was this patch tested?
Unnecessary
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.

2 participants