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

[FLINK-34671][cdc] update README.md file to update links and description. #3152

Merged
merged 15 commits into from
Mar 18, 2024

Conversation

LYanquan
Copy link
Contributor

@LYanquan LYanquan commented Mar 16, 2024

Some links of document should be updated after #3146 merged.
Rebased to master.

See https://github.com/LYanquan/flink-cdc/tree/FLINK-34671 for a preview.

@github-actions github-actions bot added the docs Improvements or additions to documentation label Mar 16, 2024
@LYanquan LYanquan force-pushed the FLINK-34671 branch 3 times, most recently from cff2403 to 33a575a Compare March 18, 2024 02:07
@LYanquan
Copy link
Contributor Author

@leonardBang @PatrickRen PTAL.

@leonardBang leonardBang self-requested a review March 18, 2024 05:36
leonardBang and others added 8 commits March 18, 2024 20:43
* 2fa/FLINK-34671:
  shell
  style
  style
  pic url
  YAML highlight
  suggestion
  suggestion
  [FLINK-34671][cdc] update README.md file to update links and description.
  [FLINK-34679][cdc][docs] Add  core concept pages for Flink CDC docs
  [FLINK-34680][cdc][docs] Add overview page for CDC connectors and optimize connector docs
  [FLINK-34682][cdc][docs] Add "Understand Flink CDC API" page for Flink CDC docs
  [FLINK-34715][cdc][cdc-connector][mysql] Correctly close BinlogSplitReader in MySQL UTCase
  [FLINK-34684][cdc][docs] Add licenses page for Flink CDC docs
  [FLINK-34683][cdc][docs] Add contributing page for Flink CDC docs
  [FLINK-34678][cdc][docs] Add introduction page for Flink CDC docs

# Conflicts:
#	README.md
* 2fa/FLINK-34671:
  suggestion

# Conflicts:
#	README.md
#	docs/static/fig/flink-cdc-logo.png
Copy link
Contributor

@leonardBang leonardBang left a comment

Choose a reason for hiding this comment

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

Thanks @LYanquan for the contribution, I also push one commit to polish the description, LGTM now

@leonardBang leonardBang merged commit a332fad into apache:master Mar 18, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants