Re: [PR] [FLINK-34362][docs] Add argument to skip integrate connector docs in setup_docs.sh to improve build times [flink]

2024-02-25 Thread via GitHub
LadyForest merged PR #24271: URL: https://github.com/apache/flink/pull/24271 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] [FLINK-34362][docs] Add argument to skip integrate connector docs in setup_docs.sh to improve build times [flink]

2024-02-25 Thread via GitHub
swuferhong commented on code in PR #24271: URL: https://github.com/apache/flink/pull/24271#discussion_r1501959629 ## docs/setup_docs.sh: ## @@ -36,24 +36,33 @@ function integrate_connector_docs { } +SKIP_INTEGRATE_CONNECTOR_DOCS=false +for arg in "$@"; do + if [ "$arg" ==

Re: [PR] [FLINK-34362][docs] Add argument to skip integrate connector docs in setup_docs.sh to improve build times [flink]

2024-02-25 Thread via GitHub
swuferhong commented on code in PR #24271: URL: https://github.com/apache/flink/pull/24271#discussion_r1501952578 ## docs/README.md: ## @@ -37,6 +37,14 @@ Then build the docs from source: $ ./build_docs.sh ``` +The shell `./build_docs.sh` will integrate external connector

Re: [PR] [FLINK-34362][docs] Add argument to skip integrate connector docs in setup_docs.sh to improve build times [flink]

2024-02-23 Thread via GitHub
LadyForest commented on code in PR #24271: URL: https://github.com/apache/flink/pull/24271#discussion_r1501360520 ## docs/setup_docs.sh: ## @@ -36,24 +36,33 @@ function integrate_connector_docs { } +SKIP_INTEGRATE_CONNECTOR_DOCS=false +for arg in "$@"; do + if [ "$arg" ==

Re: [PR] [FLINK-34362][docs] Add argument to skip integrate connector docs in setup_docs.sh to improve build times [flink]

2024-02-21 Thread via GitHub
swuferhong commented on PR #24271: URL: https://github.com/apache/flink/pull/24271#issuecomment-1956296250 Hi, @MartijnVisser. Could you please help review it again? Thank you so much. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] [FLINK-34362][docs] Add argument to skip integrate connector docs in setup_docs.sh to improve build times [flink]

2024-02-18 Thread via GitHub
swuferhong commented on code in PR #24271: URL: https://github.com/apache/flink/pull/24271#discussion_r1493723623 ## docs/setup_docs.sh: ## @@ -36,24 +36,34 @@ function integrate_connector_docs { } +SKIP_INTEGRATE_CONNECTOR_DOCS=false +for arg in "$@"; do + if [ "$arg" ==

Re: [PR] [FLINK-34362][docs] Add argument to skip integrate connector docs in setup_docs.sh to improve build times [flink]

2024-02-05 Thread via GitHub
MartijnVisser commented on code in PR #24271: URL: https://github.com/apache/flink/pull/24271#discussion_r1479341371 ## docs/setup_docs.sh: ## @@ -36,24 +36,34 @@ function integrate_connector_docs { } +SKIP_INTEGRATE_CONNECTOR_DOCS=false +for arg in "$@"; do + if [ "$arg"

Re: [PR] [FLINK-34362][docs] Add argument to skip integrate connector docs in setup_docs.sh to improve build times [flink]

2024-02-05 Thread via GitHub
flinkbot commented on PR #24271: URL: https://github.com/apache/flink/pull/24271#issuecomment-1928665414 ## CI report: * 5d0094be3680b0a36ea2ffccac1add228368b1b5 UNKNOWN Bot commands The @flinkbot bot supports the following commands: - `@flinkbot run azure`

[PR] [FLINK-34362][docs] Add argument to skip integrate connector docs in setup_docs.sh to improve build times [flink]

2024-02-05 Thread via GitHub
swuferhong opened a new pull request, #24271: URL: https://github.com/apache/flink/pull/24271 ## What is the purpose of the change The current build process of Flink's documentation involves the `setup_docs.sh` script, which re-clones connector repositories every time the