(DEPRECATED) Apache Flink Mailing List archive.

[Reminder] Prefer {% link %} tag in documentation

Classic

List

Threaded

16 messages Options

Seth Wiesman-4

[Reminder] Prefer {% link %} tag in documentation

Hi Everyone!

As we are seeing an influx of documentation PRs in anticipation of the 1.11
release I would like to remind everyone to use the {% link %} tag when
cross-linking pages[1]. This is opposed to creating a link based on
site.baseurl.

Going forward a link such as:

[CLI]({% site.baseurl %}/ops/cli.html)

Should be written as:

[CLI]({% link ops/cli.md %})

This tag will fail the build on broken links which will help us prevent
404s on the website.

You can see a good example of the link tag in action here[2].

Seth

[1] https://jekyllrb.com/docs/liquid/tags/
[2]
https://github.com/apache/flink/blame/b6ea96251d101ca25aa6a6b92170cfa4274b4cc3/docs/index.md#L65-L67

dwysakowicz

Re: [Reminder] Prefer {% link %} tag in documentation

Hi Seth,

Thanks I did not know that.

I am not entirely sure, but I think our documentation guide is slightly outdated on that manner (https://flink.apache.org/contributing/docs-style.html) Or is there a mistake in your example? Our guide recommends:

[CLI]({{ site.baseurl }}{% link ops/cli.md %})

Best,

Dawid

On 09/06/2020 16:20, Seth Wiesman wrote:

Hi Everyone!

As we are seeing an influx of documentation PRs in anticipation of the 1.11
release I would like to remind everyone to use the {% link %} tag when
cross-linking pages[1]. This is opposed to creating a link based on
site.baseurl.

Going forward a link such as:

[CLI]({% site.baseurl %}/ops/cli.html)

Should be written as:

[CLI]({% link ops/cli.md %})

This tag will fail the build on broken links which will help us prevent
404s on the website.

You can see a good example of the link tag in action here[2].

Seth

[1] https://jekyllrb.com/docs/liquid/tags/
[2]
https://github.com/apache/flink/blame/b6ea96251d101ca25aa6a6b92170cfa4274b4cc3/docs/index.md#L65-L67

signature.asc (849 bytes) Download Attachment

Seth Wiesman-3

Re: [Reminder] Prefer {% link %} tag in documentation

Thank you for noticing that, the guide is out of date. I will fix that
immediately!

On Tue, Jun 9, 2020 at 9:34 AM Dawid Wysakowicz <[hidden email]>
wrote:

> Hi Seth,
>
> Thanks I did not know that.
>
> I am not entirely sure, but I think our documentation guide is slightly
> outdated on that manner (
> https://flink.apache.org/contributing/docs-style.html) Or is there a
> mistake in your example? Our guide recommends:
>
> [CLI]({{ site.baseurl }}{% link ops/cli.md %})
>
>
> Best,
>
> Dawid
> On 09/06/2020 16:20, Seth Wiesman wrote:
>
> Hi Everyone!
>
> As we are seeing an influx of documentation PRs in anticipation of the 1.11
> release I would like to remind everyone to use the {% link %} tag when
> cross-linking pages[1]. This is opposed to creating a link based on
> site.baseurl.
>
> Going forward a link such as:
>
> [CLI]({% site.baseurl %}/ops/cli.html)
>
> Should be written as:
>
> [CLI]({% link ops/cli.md %})
>
> This tag will fail the build on broken links which will help us prevent
> 404s on the website.
>
> You can see a good example of the link tag in action here[2].
>
> Seth
>
> [1] https://jekyllrb.com/docs/liquid/tags/
> [2]https://github.com/apache/flink/blame/b6ea96251d101ca25aa6a6b92170cfa4274b4cc3/docs/index.md#L65-L67
>
>

--

Seth Wiesman | Solutions Architect

+1 314 387 1463

<https://www.ververica.com/>

Follow us @VervericaData

--

Join Flink Forward <https://flink-forward.org/> - The Apache Flink
Conference

Stream Processing | Event Driven | Real Time

Seth Wiesman-4

Re: [Reminder] Prefer {% link %} tag in documentation

Whoops, responded from the wrong email :)

Thank you for noticing that, the guide is out of date. I will fix that
immediately!

On Tue, Jun 9, 2020 at 9:36 AM Seth Wiesman <[hidden email]> wrote:

> Thank you for noticing that, the guide is out of date. I will fix that
> immediately!
>
> On Tue, Jun 9, 2020 at 9:34 AM Dawid Wysakowicz <[hidden email]>
> wrote:
>
>> Hi Seth,
>>
>> Thanks I did not know that.
>>
>> I am not entirely sure, but I think our documentation guide is slightly
>> outdated on that manner (
>> https://flink.apache.org/contributing/docs-style.html) Or is there a
>> mistake in your example? Our guide recommends:
>>
>> [CLI]({{ site.baseurl }}{% link ops/cli.md %})
>>
>>
>> Best,
>>
>> Dawid
>> On 09/06/2020 16:20, Seth Wiesman wrote:
>>
>> Hi Everyone!
>>
>> As we are seeing an influx of documentation PRs in anticipation of the 1.11
>> release I would like to remind everyone to use the {% link %} tag when
>> cross-linking pages[1]. This is opposed to creating a link based on
>> site.baseurl.
>>
>> Going forward a link such as:
>>
>> [CLI]({% site.baseurl %}/ops/cli.html)
>>
>> Should be written as:
>>
>> [CLI]({% link ops/cli.md %})
>>
>> This tag will fail the build on broken links which will help us prevent
>> 404s on the website.
>>
>> You can see a good example of the link tag in action here[2].
>>
>> Seth
>>
>> [1] https://jekyllrb.com/docs/liquid/tags/
>> [2]https://github.com/apache/flink/blame/b6ea96251d101ca25aa6a6b92170cfa4274b4cc3/docs/index.md#L65-L67
>>
>>
>
> --
>
> Seth Wiesman | Solutions Architect
>
> +1 314 387 1463
>
> <https://www.ververica.com/>
>
> Follow us @VervericaData
>
> --
>
> Join Flink Forward <https://flink-forward.org/> - The Apache Flink
> Conference
>
> Stream Processing | Event Driven | Real Time
>

Robert Metzger

Re: [Reminder] Prefer {% link %} tag in documentation

Thanks for the reminder. I was also not aware of this tag!

How about enforcing the use of this tag through CI?
We could for example grep through the added lines of all changes in docs/
and fail the build if we see the wrong pattern.

On Tue, Jun 9, 2020 at 4:37 PM Seth Wiesman <[hidden email]> wrote:

> Whoops, responded from the wrong email :)
>
> Thank you for noticing that, the guide is out of date. I will fix that
> immediately!
>
> On Tue, Jun 9, 2020 at 9:36 AM Seth Wiesman <[hidden email]> wrote:
>
> > Thank you for noticing that, the guide is out of date. I will fix that
> > immediately!
> >
> > On Tue, Jun 9, 2020 at 9:34 AM Dawid Wysakowicz <[hidden email]>
> > wrote:
> >
> >> Hi Seth,
> >>
> >> Thanks I did not know that.
> >>
> >> I am not entirely sure, but I think our documentation guide is slightly
> >> outdated on that manner (
> >> https://flink.apache.org/contributing/docs-style.html) Or is there a
> >> mistake in your example? Our guide recommends:
> >>
> >> [CLI]({{ site.baseurl }}{% link ops/cli.md %})
> >>
> >>
> >> Best,
> >>
> >> Dawid
> >> On 09/06/2020 16:20, Seth Wiesman wrote:
> >>
> >> Hi Everyone!
> >>
> >> As we are seeing an influx of documentation PRs in anticipation of the
> 1.11
> >> release I would like to remind everyone to use the {% link %} tag when
> >> cross-linking pages[1]. This is opposed to creating a link based on
> >> site.baseurl.
> >>
> >> Going forward a link such as:
> >>
> >> [CLI]({% site.baseurl %}/ops/cli.html)
> >>
> >> Should be written as:
> >>
> >> [CLI]({% link ops/cli.md %})
> >>
> >> This tag will fail the build on broken links which will help us prevent
> >> 404s on the website.
> >>
> >> You can see a good example of the link tag in action here[2].
> >>
> >> Seth
> >>
> >> [1] https://jekyllrb.com/docs/liquid/tags/
> >> [2]
> https://github.com/apache/flink/blame/b6ea96251d101ca25aa6a6b92170cfa4274b4cc3/docs/index.md#L65-L67
> >>
> >>
> >
> > --
> >
> > Seth Wiesman | Solutions Architect
> >
> > +1 314 387 1463
> >
> > <https://www.ververica.com/>
> >
> > Follow us @VervericaData
> >
> > --
> >
> > Join Flink Forward <https://flink-forward.org/> - The Apache Flink
> > Conference
> >
> > Stream Processing | Event Driven | Real Time
> >
>

Seth Wiesman-4

Re: [Reminder] Prefer {% link %} tag in documentation

The tag is new to Jekyll 4.0 which we only recently updated to.

There are a lot of existing tags that would need to be updated first :)
I opened a ticket to track that work and then yes that would make sense.

Seth

[1] https://issues.apache.org/jira/browse/FLINK-18193

On Tue, Jun 9, 2020 at 12:05 PM Robert Metzger <[hidden email]> wrote:

> Thanks for the reminder. I was also not aware of this tag!
>
> How about enforcing the use of this tag through CI?
> We could for example grep through the added lines of all changes in docs/
> and fail the build if we see the wrong pattern.
>
>
> On Tue, Jun 9, 2020 at 4:37 PM Seth Wiesman <[hidden email]> wrote:
>
> > Whoops, responded from the wrong email :)
> >
> > Thank you for noticing that, the guide is out of date. I will fix that
> > immediately!
> >
> > On Tue, Jun 9, 2020 at 9:36 AM Seth Wiesman <[hidden email]> wrote:
> >
> > > Thank you for noticing that, the guide is out of date. I will fix that
> > > immediately!
> > >
> > > On Tue, Jun 9, 2020 at 9:34 AM Dawid Wysakowicz <
> [hidden email]>
> > > wrote:
> > >
> > >> Hi Seth,
> > >>
> > >> Thanks I did not know that.
> > >>
> > >> I am not entirely sure, but I think our documentation guide is
> slightly
> > >> outdated on that manner (
> > >> https://flink.apache.org/contributing/docs-style.html) Or is there a
> > >> mistake in your example? Our guide recommends:
> > >>
> > >> [CLI]({{ site.baseurl }}{% link ops/cli.md %})
> > >>
> > >>
> > >> Best,
> > >>
> > >> Dawid
> > >> On 09/06/2020 16:20, Seth Wiesman wrote:
> > >>
> > >> Hi Everyone!
> > >>
> > >> As we are seeing an influx of documentation PRs in anticipation of the
> > 1.11
> > >> release I would like to remind everyone to use the {% link %} tag when
> > >> cross-linking pages[1]. This is opposed to creating a link based on
> > >> site.baseurl.
> > >>
> > >> Going forward a link such as:
> > >>
> > >> [CLI]({% site.baseurl %}/ops/cli.html)
> > >>
> > >> Should be written as:
> > >>
> > >> [CLI]({% link ops/cli.md %})
> > >>
> > >> This tag will fail the build on broken links which will help us
> prevent
> > >> 404s on the website.
> > >>
> > >> You can see a good example of the link tag in action here[2].
> > >>
> > >> Seth
> > >>
> > >> [1] https://jekyllrb.com/docs/liquid/tags/
> > >> [2]
> >
> https://github.com/apache/flink/blame/b6ea96251d101ca25aa6a6b92170cfa4274b4cc3/docs/index.md#L65-L67
> > >>
> > >>
> > >
> > > --
> > >
> > > Seth Wiesman | Solutions Architect
> > >
> > > +1 314 387 1463
> > >
> > > <https://www.ververica.com/>
> > >
> > > Follow us @VervericaData
> > >
> > > --
> > >
> > > Join Flink Forward <https://flink-forward.org/> - The Apache Flink
> > > Conference
> > >
> > > Stream Processing | Event Driven | Real Time
> > >
> >
>

Kostas Kloudas-4

Re: [Reminder] Prefer {% link %} tag in documentation

Thanks for the heads up Seth!

Kostas

On Tue, Jun 9, 2020 at 7:27 PM Seth Wiesman <[hidden email]> wrote:

>
> The tag is new to Jekyll 4.0 which we only recently updated to.
>
> There are a lot of existing tags that would need to be updated first :)
> I opened a ticket to track that work and then yes that would make sense.
>
> Seth
>
> [1] https://issues.apache.org/jira/browse/FLINK-18193
>
> On Tue, Jun 9, 2020 at 12:05 PM Robert Metzger <[hidden email]> wrote:
>
> > Thanks for the reminder. I was also not aware of this tag!
> >
> > How about enforcing the use of this tag through CI?
> > We could for example grep through the added lines of all changes in docs/
> > and fail the build if we see the wrong pattern.
> >
> >
> > On Tue, Jun 9, 2020 at 4:37 PM Seth Wiesman <[hidden email]> wrote:
> >
> > > Whoops, responded from the wrong email :)
> > >
> > > Thank you for noticing that, the guide is out of date. I will fix that
> > > immediately!
> > >
> > > On Tue, Jun 9, 2020 at 9:36 AM Seth Wiesman <[hidden email]> wrote:
> > >
> > > > Thank you for noticing that, the guide is out of date. I will fix that
> > > > immediately!
> > > >
> > > > On Tue, Jun 9, 2020 at 9:34 AM Dawid Wysakowicz <
> > [hidden email]>
> > > > wrote:
> > > >
> > > >> Hi Seth,
> > > >>
> > > >> Thanks I did not know that.
> > > >>
> > > >> I am not entirely sure, but I think our documentation guide is
> > slightly
> > > >> outdated on that manner (
> > > >> https://flink.apache.org/contributing/docs-style.html) Or is there a
> > > >> mistake in your example? Our guide recommends:
> > > >>
> > > >> [CLI]({{ site.baseurl }}{% link ops/cli.md %})
> > > >>
> > > >>
> > > >> Best,
> > > >>
> > > >> Dawid
> > > >> On 09/06/2020 16:20, Seth Wiesman wrote:
> > > >>
> > > >> Hi Everyone!
> > > >>
> > > >> As we are seeing an influx of documentation PRs in anticipation of the
> > > 1.11
> > > >> release I would like to remind everyone to use the {% link %} tag when
> > > >> cross-linking pages[1]. This is opposed to creating a link based on
> > > >> site.baseurl.
> > > >>
> > > >> Going forward a link such as:
> > > >>
> > > >> [CLI]({% site.baseurl %}/ops/cli.html)
> > > >>
> > > >> Should be written as:
> > > >>
> > > >> [CLI]({% link ops/cli.md %})
> > > >>
> > > >> This tag will fail the build on broken links which will help us
> > prevent
> > > >> 404s on the website.
> > > >>
> > > >> You can see a good example of the link tag in action here[2].
> > > >>
> > > >> Seth
> > > >>
> > > >> [1] https://jekyllrb.com/docs/liquid/tags/
> > > >> [2]
> > >
> > https://github.com/apache/flink/blame/b6ea96251d101ca25aa6a6b92170cfa4274b4cc3/docs/index.md#L65-L67
> > > >>
> > > >>
> > > >
> > > > --
> > > >
> > > > Seth Wiesman | Solutions Architect
> > > >
> > > > +1 314 387 1463
> > > >
> > > > <https://www.ververica.com/>
> > > >
> > > > Follow us @VervericaData
> > > >
> > > > --
> > > >
> > > > Join Flink Forward <https://flink-forward.org/> - The Apache Flink
> > > > Conference
> > > >
> > > > Stream Processing | Event Driven | Real Time
> > > >
> > >
> >

Yangze Guo

Re: [Reminder] Prefer {% link %} tag in documentation

Thanks for that reminder, Seth!

+1 to add a check during CI if possible.

Best,
Yangze Guo

On Wed, Jun 10, 2020 at 3:04 AM Kostas Kloudas <[hidden email]> wrote:

>
> Thanks for the heads up Seth!
>
> Kostas
>
> On Tue, Jun 9, 2020 at 7:27 PM Seth Wiesman <[hidden email]> wrote:
> >
> > The tag is new to Jekyll 4.0 which we only recently updated to.
> >
> > There are a lot of existing tags that would need to be updated first :)
> > I opened a ticket to track that work and then yes that would make sense.
> >
> > Seth
> >
> > [1] https://issues.apache.org/jira/browse/FLINK-18193
> >
> > On Tue, Jun 9, 2020 at 12:05 PM Robert Metzger <[hidden email]> wrote:
> >
> > > Thanks for the reminder. I was also not aware of this tag!
> > >
> > > How about enforcing the use of this tag through CI?
> > > We could for example grep through the added lines of all changes in docs/
> > > and fail the build if we see the wrong pattern.
> > >
> > >
> > > On Tue, Jun 9, 2020 at 4:37 PM Seth Wiesman <[hidden email]> wrote:
> > >
> > > > Whoops, responded from the wrong email :)
> > > >
> > > > Thank you for noticing that, the guide is out of date. I will fix that
> > > > immediately!
> > > >
> > > > On Tue, Jun 9, 2020 at 9:36 AM Seth Wiesman <[hidden email]> wrote:
> > > >
> > > > > Thank you for noticing that, the guide is out of date. I will fix that
> > > > > immediately!
> > > > >
> > > > > On Tue, Jun 9, 2020 at 9:34 AM Dawid Wysakowicz <
> > > [hidden email]>
> > > > > wrote:
> > > > >
> > > > >> Hi Seth,
> > > > >>
> > > > >> Thanks I did not know that.
> > > > >>
> > > > >> I am not entirely sure, but I think our documentation guide is
> > > slightly
> > > > >> outdated on that manner (
> > > > >> https://flink.apache.org/contributing/docs-style.html) Or is there a
> > > > >> mistake in your example? Our guide recommends:
> > > > >>
> > > > >> [CLI]({{ site.baseurl }}{% link ops/cli.md %})
> > > > >>
> > > > >>
> > > > >> Best,
> > > > >>
> > > > >> Dawid
> > > > >> On 09/06/2020 16:20, Seth Wiesman wrote:
> > > > >>
> > > > >> Hi Everyone!
> > > > >>
> > > > >> As we are seeing an influx of documentation PRs in anticipation of the
> > > > 1.11
> > > > >> release I would like to remind everyone to use the {% link %} tag when
> > > > >> cross-linking pages[1]. This is opposed to creating a link based on
> > > > >> site.baseurl.
> > > > >>
> > > > >> Going forward a link such as:
> > > > >>
> > > > >> [CLI]({% site.baseurl %}/ops/cli.html)
> > > > >>
> > > > >> Should be written as:
> > > > >>
> > > > >> [CLI]({% link ops/cli.md %})
> > > > >>
> > > > >> This tag will fail the build on broken links which will help us
> > > prevent
> > > > >> 404s on the website.
> > > > >>
> > > > >> You can see a good example of the link tag in action here[2].
> > > > >>
> > > > >> Seth
> > > > >>
> > > > >> [1] https://jekyllrb.com/docs/liquid/tags/
> > > > >> [2]
> > > >
> > > https://github.com/apache/flink/blame/b6ea96251d101ca25aa6a6b92170cfa4274b4cc3/docs/index.md#L65-L67
> > > > >>
> > > > >>
> > > > >
> > > > > --
> > > > >
> > > > > Seth Wiesman | Solutions Architect
> > > > >
> > > > > +1 314 387 1463
> > > > >
> > > > > <https://www.ververica.com/>
> > > > >
> > > > > Follow us @VervericaData
> > > > >
> > > > > --
> > > > >
> > > > > Join Flink Forward <https://flink-forward.org/> - The Apache Flink
> > > > > Conference
> > > > >
> > > > > Stream Processing | Event Driven | Real Time
> > > > >
> > > >
> > >

Jark Wu-2

Re: [Reminder] Prefer {% link %} tag in documentation

+1 to use {% link %} tag and add check in CI.

Tips: if want to link a Chinese page, should write: [CLI]({% link ops/
cli.zh.md %})

Best,
Jark

On Wed, 10 Jun 2020 at 10:30, Yangze Guo <[hidden email]> wrote:

> Thanks for that reminder, Seth!
>
> +1 to add a check during CI if possible.
>
> Best,
> Yangze Guo
>
> On Wed, Jun 10, 2020 at 3:04 AM Kostas Kloudas <[hidden email]> wrote:
> >
> > Thanks for the heads up Seth!
> >
> > Kostas
> >
> > On Tue, Jun 9, 2020 at 7:27 PM Seth Wiesman <[hidden email]> wrote:
> > >
> > > The tag is new to Jekyll 4.0 which we only recently updated to.
> > >
> > > There are a lot of existing tags that would need to be updated first :)
> > > I opened a ticket to track that work and then yes that would make
> sense.
> > >
> > > Seth
> > >
> > > [1] https://issues.apache.org/jira/browse/FLINK-18193
> > >
> > > On Tue, Jun 9, 2020 at 12:05 PM Robert Metzger <[hidden email]>
> wrote:
> > >
> > > > Thanks for the reminder. I was also not aware of this tag!
> > > >
> > > > How about enforcing the use of this tag through CI?
> > > > We could for example grep through the added lines of all changes in
> docs/
> > > > and fail the build if we see the wrong pattern.
> > > >
> > > >
> > > > On Tue, Jun 9, 2020 at 4:37 PM Seth Wiesman <[hidden email]>
> wrote:
> > > >
> > > > > Whoops, responded from the wrong email :)
> > > > >
> > > > > Thank you for noticing that, the guide is out of date. I will fix
> that
> > > > > immediately!
> > > > >
> > > > > On Tue, Jun 9, 2020 at 9:36 AM Seth Wiesman <[hidden email]>
> wrote:
> > > > >
> > > > > > Thank you for noticing that, the guide is out of date. I will
> fix that
> > > > > > immediately!
> > > > > >
> > > > > > On Tue, Jun 9, 2020 at 9:34 AM Dawid Wysakowicz <
> > > > [hidden email]>
> > > > > > wrote:
> > > > > >
> > > > > >> Hi Seth,
> > > > > >>
> > > > > >> Thanks I did not know that.
> > > > > >>
> > > > > >> I am not entirely sure, but I think our documentation guide is
> > > > slightly
> > > > > >> outdated on that manner (
> > > > > >> https://flink.apache.org/contributing/docs-style.html) Or is
> there a
> > > > > >> mistake in your example? Our guide recommends:
> > > > > >>
> > > > > >> [CLI]({{ site.baseurl }}{% link ops/cli.md %})
> > > > > >>
> > > > > >>
> > > > > >> Best,
> > > > > >>
> > > > > >> Dawid
> > > > > >> On 09/06/2020 16:20, Seth Wiesman wrote:
> > > > > >>
> > > > > >> Hi Everyone!
> > > > > >>
> > > > > >> As we are seeing an influx of documentation PRs in anticipation
> of the
> > > > > 1.11
> > > > > >> release I would like to remind everyone to use the {% link %}
> tag when
> > > > > >> cross-linking pages[1]. This is opposed to creating a link
> based on
> > > > > >> site.baseurl.
> > > > > >>
> > > > > >> Going forward a link such as:
> > > > > >>
> > > > > >> [CLI]({% site.baseurl %}/ops/cli.html)
> > > > > >>
> > > > > >> Should be written as:
> > > > > >>
> > > > > >> [CLI]({% link ops/cli.md %})
> > > > > >>
> > > > > >> This tag will fail the build on broken links which will help us
> > > > prevent
> > > > > >> 404s on the website.
> > > > > >>
> > > > > >> You can see a good example of the link tag in action here[2].
> > > > > >>
> > > > > >> Seth
> > > > > >>
> > > > > >> [1] https://jekyllrb.com/docs/liquid/tags/
> > > > > >> [2]
> > > > >
> > > >
> https://github.com/apache/flink/blame/b6ea96251d101ca25aa6a6b92170cfa4274b4cc3/docs/index.md#L65-L67
> > > > > >>
> > > > > >>
> > > > > >
> > > > > > --
> > > > > >
> > > > > > Seth Wiesman | Solutions Architect
> > > > > >
> > > > > > +1 314 387 1463
> > > > > >
> > > > > > <https://www.ververica.com/>
> > > > > >
> > > > > > Follow us @VervericaData
> > > > > >
> > > > > > --
> > > > > >
> > > > > > Join Flink Forward <https://flink-forward.org/> - The Apache
> Flink
> > > > > > Conference
> > > > > >
> > > > > > Stream Processing | Event Driven | Real Time
> > > > > >
> > > > >
> > > >
>

Congxian Qiu

Re: [Reminder] Prefer {% link %} tag in documentation

+1 to use {% link %} tag and add a check during CI.
for Chinese doc, will suggest the Chinese translate contributor use the {%
link %} tag when reviewing the translate pr.

Best,
Congxian

Jark Wu <[hidden email]> 于2020年6月10日周三上午10:48写道：

> +1 to use {% link %} tag and add check in CI.
>
> Tips: if want to link a Chinese page, should write: [CLI]({% link ops/
> cli.zh.md %})
>
> Best,
> Jark
>
> On Wed, 10 Jun 2020 at 10:30, Yangze Guo <[hidden email]> wrote:
>
> > Thanks for that reminder, Seth!
> >
> > +1 to add a check during CI if possible.
> >
> > Best,
> > Yangze Guo
> >
> > On Wed, Jun 10, 2020 at 3:04 AM Kostas Kloudas <[hidden email]>
> wrote:
> > >
> > > Thanks for the heads up Seth!
> > >
> > > Kostas
> > >
> > > On Tue, Jun 9, 2020 at 7:27 PM Seth Wiesman <[hidden email]>
> wrote:
> > > >
> > > > The tag is new to Jekyll 4.0 which we only recently updated to.
> > > >
> > > > There are a lot of existing tags that would need to be updated first
> :)
> > > > I opened a ticket to track that work and then yes that would make
> > sense.
> > > >
> > > > Seth
> > > >
> > > > [1] https://issues.apache.org/jira/browse/FLINK-18193
> > > >
> > > > On Tue, Jun 9, 2020 at 12:05 PM Robert Metzger <[hidden email]>
> > wrote:
> > > >
> > > > > Thanks for the reminder. I was also not aware of this tag!
> > > > >
> > > > > How about enforcing the use of this tag through CI?
> > > > > We could for example grep through the added lines of all changes in
> > docs/
> > > > > and fail the build if we see the wrong pattern.
> > > > >
> > > > >
> > > > > On Tue, Jun 9, 2020 at 4:37 PM Seth Wiesman <[hidden email]>
> > wrote:
> > > > >
> > > > > > Whoops, responded from the wrong email :)
> > > > > >
> > > > > > Thank you for noticing that, the guide is out of date. I will fix
> > that
> > > > > > immediately!
> > > > > >
> > > > > > On Tue, Jun 9, 2020 at 9:36 AM Seth Wiesman <[hidden email]>
> > wrote:
> > > > > >
> > > > > > > Thank you for noticing that, the guide is out of date. I will
> > fix that
> > > > > > > immediately!
> > > > > > >
> > > > > > > On Tue, Jun 9, 2020 at 9:34 AM Dawid Wysakowicz <
> > > > > [hidden email]>
> > > > > > > wrote:
> > > > > > >
> > > > > > >> Hi Seth,
> > > > > > >>
> > > > > > >> Thanks I did not know that.
> > > > > > >>
> > > > > > >> I am not entirely sure, but I think our documentation guide is
> > > > > slightly
> > > > > > >> outdated on that manner (
> > > > > > >> https://flink.apache.org/contributing/docs-style.html) Or is
> > there a
> > > > > > >> mistake in your example? Our guide recommends:
> > > > > > >>
> > > > > > >> [CLI]({{ site.baseurl }}{% link ops/cli.md %})
> > > > > > >>
> > > > > > >>
> > > > > > >> Best,
> > > > > > >>
> > > > > > >> Dawid
> > > > > > >> On 09/06/2020 16:20, Seth Wiesman wrote:
> > > > > > >>
> > > > > > >> Hi Everyone!
> > > > > > >>
> > > > > > >> As we are seeing an influx of documentation PRs in
> anticipation
> > of the
> > > > > > 1.11
> > > > > > >> release I would like to remind everyone to use the {% link %}
> > tag when
> > > > > > >> cross-linking pages[1]. This is opposed to creating a link
> > based on
> > > > > > >> site.baseurl.
> > > > > > >>
> > > > > > >> Going forward a link such as:
> > > > > > >>
> > > > > > >> [CLI]({% site.baseurl %}/ops/cli.html)
> > > > > > >>
> > > > > > >> Should be written as:
> > > > > > >>
> > > > > > >> [CLI]({% link ops/cli.md %})
> > > > > > >>
> > > > > > >> This tag will fail the build on broken links which will help
> us
> > > > > prevent
> > > > > > >> 404s on the website.
> > > > > > >>
> > > > > > >> You can see a good example of the link tag in action here[2].
> > > > > > >>
> > > > > > >> Seth
> > > > > > >>
> > > > > > >> [1] https://jekyllrb.com/docs/liquid/tags/
> > > > > > >> [2]
> > > > > >
> > > > >
> >
> https://github.com/apache/flink/blame/b6ea96251d101ca25aa6a6b92170cfa4274b4cc3/docs/index.md#L65-L67
> > > > > > >>
> > > > > > >>
> > > > > > >
> > > > > > > --
> > > > > > >
> > > > > > > Seth Wiesman | Solutions Architect
> > > > > > >
> > > > > > > +1 314 387 1463
> > > > > > >
> > > > > > > <https://www.ververica.com/>
> > > > > > >
> > > > > > > Follow us @VervericaData
> > > > > > >
> > > > > > > --
> > > > > > >
> > > > > > > Join Flink Forward <https://flink-forward.org/> - The Apache
> > Flink
> > > > > > > Conference
> > > > > > >
> > > > > > > Stream Processing | Event Driven | Real Time
> > > > > > >
> > > > > >
> > > > >
> >
>

Jark Wu-2

Re: [Reminder] Prefer {% link %} tag in documentation

Hi everyone,

Before going to use {% link %} and adding a CI to force using link tag, I
suggest adding a CI profile to check broken links first.

The background is that, recently, I noticed that many contributors are
beginning to use link tags, but forget to link ".zh.md" instead of ".md" in
Chinese documentation.
This leads to the docs build failing in the last two days [1]. I have fixed
a couple of broken links. But if we don't have a CI mechanism, this would
make the docs build unstable.

Best,
Jark

[1]: https://ci.apache.org/builders/flink-docs-master

On Mon, 15 Jun 2020 at 12:48, Congxian Qiu <[hidden email]> wrote:

> +1 to use {% link %} tag and add a check during CI.
> for Chinese doc, will suggest the Chinese translate contributor use the {%
> link %} tag when reviewing the translate pr.
>
> Best,
> Congxian
>
>
> Jark Wu <[hidden email]> 于2020年6月10日周三上午10:48写道：
>
> > +1 to use {% link %} tag and add check in CI.
> >
> > Tips: if want to link a Chinese page, should write: [CLI]({% link ops/
> > cli.zh.md %})
> >
> > Best,
> > Jark
> >
> > On Wed, 10 Jun 2020 at 10:30, Yangze Guo <[hidden email]> wrote:
> >
> > > Thanks for that reminder, Seth!
> > >
> > > +1 to add a check during CI if possible.
> > >
> > > Best,
> > > Yangze Guo
> > >
> > > On Wed, Jun 10, 2020 at 3:04 AM Kostas Kloudas <[hidden email]>
> > wrote:
> > > >
> > > > Thanks for the heads up Seth!
> > > >
> > > > Kostas
> > > >
> > > > On Tue, Jun 9, 2020 at 7:27 PM Seth Wiesman <[hidden email]>
> > wrote:
> > > > >
> > > > > The tag is new to Jekyll 4.0 which we only recently updated to.
> > > > >
> > > > > There are a lot of existing tags that would need to be updated
> first
> > :)
> > > > > I opened a ticket to track that work and then yes that would make
> > > sense.
> > > > >
> > > > > Seth
> > > > >
> > > > > [1] https://issues.apache.org/jira/browse/FLINK-18193
> > > > >
> > > > > On Tue, Jun 9, 2020 at 12:05 PM Robert Metzger <
> [hidden email]>
> > > wrote:
> > > > >
> > > > > > Thanks for the reminder. I was also not aware of this tag!
> > > > > >
> > > > > > How about enforcing the use of this tag through CI?
> > > > > > We could for example grep through the added lines of all changes
> in
> > > docs/
> > > > > > and fail the build if we see the wrong pattern.
> > > > > >
> > > > > >
> > > > > > On Tue, Jun 9, 2020 at 4:37 PM Seth Wiesman <[hidden email]
> >
> > > wrote:
> > > > > >
> > > > > > > Whoops, responded from the wrong email :)
> > > > > > >
> > > > > > > Thank you for noticing that, the guide is out of date. I will
> fix
> > > that
> > > > > > > immediately!
> > > > > > >
> > > > > > > On Tue, Jun 9, 2020 at 9:36 AM Seth Wiesman <
> [hidden email]>
> > > wrote:
> > > > > > >
> > > > > > > > Thank you for noticing that, the guide is out of date. I will
> > > fix that
> > > > > > > > immediately!
> > > > > > > >
> > > > > > > > On Tue, Jun 9, 2020 at 9:34 AM Dawid Wysakowicz <
> > > > > > [hidden email]>
> > > > > > > > wrote:
> > > > > > > >
> > > > > > > >> Hi Seth,
> > > > > > > >>
> > > > > > > >> Thanks I did not know that.
> > > > > > > >>
> > > > > > > >> I am not entirely sure, but I think our documentation guide
> is
> > > > > > slightly
> > > > > > > >> outdated on that manner (
> > > > > > > >> https://flink.apache.org/contributing/docs-style.html) Or
> is
> > > there a
> > > > > > > >> mistake in your example? Our guide recommends:
> > > > > > > >>
> > > > > > > >> [CLI]({{ site.baseurl }}{% link ops/cli.md %})
> > > > > > > >>
> > > > > > > >>
> > > > > > > >> Best,
> > > > > > > >>
> > > > > > > >> Dawid
> > > > > > > >> On 09/06/2020 16:20, Seth Wiesman wrote:
> > > > > > > >>
> > > > > > > >> Hi Everyone!
> > > > > > > >>
> > > > > > > >> As we are seeing an influx of documentation PRs in
> > anticipation
> > > of the
> > > > > > > 1.11
> > > > > > > >> release I would like to remind everyone to use the {% link
> %}
> > > tag when
> > > > > > > >> cross-linking pages[1]. This is opposed to creating a link
> > > based on
> > > > > > > >> site.baseurl.
> > > > > > > >>
> > > > > > > >> Going forward a link such as:
> > > > > > > >>
> > > > > > > >> [CLI]({% site.baseurl %}/ops/cli.html)
> > > > > > > >>
> > > > > > > >> Should be written as:
> > > > > > > >>
> > > > > > > >> [CLI]({% link ops/cli.md %})
> > > > > > > >>
> > > > > > > >> This tag will fail the build on broken links which will help
> > us
> > > > > > prevent
> > > > > > > >> 404s on the website.
> > > > > > > >>
> > > > > > > >> You can see a good example of the link tag in action
> here[2].
> > > > > > > >>
> > > > > > > >> Seth
> > > > > > > >>
> > > > > > > >> [1] https://jekyllrb.com/docs/liquid/tags/
> > > > > > > >> [2]
> > > > > > >
> > > > > >
> > >
> >
> https://github.com/apache/flink/blame/b6ea96251d101ca25aa6a6b92170cfa4274b4cc3/docs/index.md#L65-L67
> > > > > > > >>
> > > > > > > >>
> > > > > > > >
> > > > > > > > --
> > > > > > > >
> > > > > > > > Seth Wiesman | Solutions Architect
> > > > > > > >
> > > > > > > > +1 314 387 1463
> > > > > > > >
> > > > > > > > <https://www.ververica.com/>
> > > > > > > >
> > > > > > > > Follow us @VervericaData
> > > > > > > >
> > > > > > > > --
> > > > > > > >
> > > > > > > > Join Flink Forward <https://flink-forward.org/> - The Apache
> > > Flink
> > > > > > > > Conference
> > > > > > > >
> > > > > > > > Stream Processing | Event Driven | Real Time
> > > > > > > >
> > > > > > >
> > > > > >
> > >
> >
>

Yangze Guo

Re: [Reminder] Prefer {% link %} tag in documentation

I second Jark's comment. We need a CI mechanism to ensure this.

Best,
Yangze Guo

On Wed, Jun 17, 2020 at 11:45 AM Jark Wu <[hidden email]> wrote:

>
> Hi everyone,
>
> Before going to use {% link %} and adding a CI to force using link tag, I
> suggest adding a CI profile to check broken links first.
>
> The background is that, recently, I noticed that many contributors are
> beginning to use link tags, but forget to link ".zh.md" instead of ".md" in
> Chinese documentation.
> This leads to the docs build failing in the last two days [1]. I have fixed
> a couple of broken links. But if we don't have a CI mechanism, this would
> make the docs build unstable.
>
> Best,
> Jark
>
> [1]: https://ci.apache.org/builders/flink-docs-master
>
> On Mon, 15 Jun 2020 at 12:48, Congxian Qiu <[hidden email]> wrote:
>
> > +1 to use {% link %} tag and add a check during CI.
> > for Chinese doc, will suggest the Chinese translate contributor use the {%
> > link %} tag when reviewing the translate pr.
> >
> > Best,
> > Congxian
> >
> >
> > Jark Wu <[hidden email]> 于2020年6月10日周三上午10:48写道：
> >
> > > +1 to use {% link %} tag and add check in CI.
> > >
> > > Tips: if want to link a Chinese page, should write: [CLI]({% link ops/
> > > cli.zh.md %})
> > >
> > > Best,
> > > Jark
> > >
> > > On Wed, 10 Jun 2020 at 10:30, Yangze Guo <[hidden email]> wrote:
> > >
> > > > Thanks for that reminder, Seth!
> > > >
> > > > +1 to add a check during CI if possible.
> > > >
> > > > Best,
> > > > Yangze Guo
> > > >
> > > > On Wed, Jun 10, 2020 at 3:04 AM Kostas Kloudas <[hidden email]>
> > > wrote:
> > > > >
> > > > > Thanks for the heads up Seth!
> > > > >
> > > > > Kostas
> > > > >
> > > > > On Tue, Jun 9, 2020 at 7:27 PM Seth Wiesman <[hidden email]>
> > > wrote:
> > > > > >
> > > > > > The tag is new to Jekyll 4.0 which we only recently updated to.
> > > > > >
> > > > > > There are a lot of existing tags that would need to be updated
> > first
> > > :)
> > > > > > I opened a ticket to track that work and then yes that would make
> > > > sense.
> > > > > >
> > > > > > Seth
> > > > > >
> > > > > > [1] https://issues.apache.org/jira/browse/FLINK-18193
> > > > > >
> > > > > > On Tue, Jun 9, 2020 at 12:05 PM Robert Metzger <
> > [hidden email]>
> > > > wrote:
> > > > > >
> > > > > > > Thanks for the reminder. I was also not aware of this tag!
> > > > > > >
> > > > > > > How about enforcing the use of this tag through CI?
> > > > > > > We could for example grep through the added lines of all changes
> > in
> > > > docs/
> > > > > > > and fail the build if we see the wrong pattern.
> > > > > > >
> > > > > > >
> > > > > > > On Tue, Jun 9, 2020 at 4:37 PM Seth Wiesman <[hidden email]
> > >
> > > > wrote:
> > > > > > >
> > > > > > > > Whoops, responded from the wrong email :)
> > > > > > > >
> > > > > > > > Thank you for noticing that, the guide is out of date. I will
> > fix
> > > > that
> > > > > > > > immediately!
> > > > > > > >
> > > > > > > > On Tue, Jun 9, 2020 at 9:36 AM Seth Wiesman <
> > [hidden email]>
> > > > wrote:
> > > > > > > >
> > > > > > > > > Thank you for noticing that, the guide is out of date. I will
> > > > fix that
> > > > > > > > > immediately!
> > > > > > > > >
> > > > > > > > > On Tue, Jun 9, 2020 at 9:34 AM Dawid Wysakowicz <
> > > > > > > [hidden email]>
> > > > > > > > > wrote:
> > > > > > > > >
> > > > > > > > >> Hi Seth,
> > > > > > > > >>
> > > > > > > > >> Thanks I did not know that.
> > > > > > > > >>
> > > > > > > > >> I am not entirely sure, but I think our documentation guide
> > is
> > > > > > > slightly
> > > > > > > > >> outdated on that manner (
> > > > > > > > >> https://flink.apache.org/contributing/docs-style.html) Or
> > is
> > > > there a
> > > > > > > > >> mistake in your example? Our guide recommends:
> > > > > > > > >>
> > > > > > > > >> [CLI]({{ site.baseurl }}{% link ops/cli.md %})
> > > > > > > > >>
> > > > > > > > >>
> > > > > > > > >> Best,
> > > > > > > > >>
> > > > > > > > >> Dawid
> > > > > > > > >> On 09/06/2020 16:20, Seth Wiesman wrote:
> > > > > > > > >>
> > > > > > > > >> Hi Everyone!
> > > > > > > > >>
> > > > > > > > >> As we are seeing an influx of documentation PRs in
> > > anticipation
> > > > of the
> > > > > > > > 1.11
> > > > > > > > >> release I would like to remind everyone to use the {% link
> > %}
> > > > tag when
> > > > > > > > >> cross-linking pages[1]. This is opposed to creating a link
> > > > based on
> > > > > > > > >> site.baseurl.
> > > > > > > > >>
> > > > > > > > >> Going forward a link such as:
> > > > > > > > >>
> > > > > > > > >> [CLI]({% site.baseurl %}/ops/cli.html)
> > > > > > > > >>
> > > > > > > > >> Should be written as:
> > > > > > > > >>
> > > > > > > > >> [CLI]({% link ops/cli.md %})
> > > > > > > > >>
> > > > > > > > >> This tag will fail the build on broken links which will help
> > > us
> > > > > > > prevent
> > > > > > > > >> 404s on the website.
> > > > > > > > >>
> > > > > > > > >> You can see a good example of the link tag in action
> > here[2].
> > > > > > > > >>
> > > > > > > > >> Seth
> > > > > > > > >>
> > > > > > > > >> [1] https://jekyllrb.com/docs/liquid/tags/
> > > > > > > > >> [2]
> > > > > > > >
> > > > > > >
> > > >
> > >
> > https://github.com/apache/flink/blame/b6ea96251d101ca25aa6a6b92170cfa4274b4cc3/docs/index.md#L65-L67
> > > > > > > > >>
> > > > > > > > >>
> > > > > > > > >
> > > > > > > > > --
> > > > > > > > >
> > > > > > > > > Seth Wiesman | Solutions Architect
> > > > > > > > >
> > > > > > > > > +1 314 387 1463
> > > > > > > > >
> > > > > > > > > <https://www.ververica.com/>
> > > > > > > > >
> > > > > > > > > Follow us @VervericaData
> > > > > > > > >
> > > > > > > > > --
> > > > > > > > >
> > > > > > > > > Join Flink Forward <https://flink-forward.org/> - The Apache
> > > > Flink
> > > > > > > > > Conference
> > > > > > > > >
> > > > > > > > > Stream Processing | Event Driven | Real Time
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > >
> > >
> >

RocMarshal

Re:Re: [Reminder] Prefer {% link %} tag in documentation

Hi everyone,

There is a question about optimizing and translating documents. If the link target is an internal image, does the link method need to be set as the same way that link target is other pages.
Such as : Should """ src="{{ site.baseurl }}/fig/stream_barriers.svg" """ be written as """ src="{% link fig/stream_barriers.svg %} """ ?

Thank you.
Best Roc.

At 2020-06-17 12:38:38, "Yangze Guo" <[hidden email]> wrote:

>I second Jark's comment. We need a CI mechanism to ensure this.
>
>Best,
>Yangze Guo
>
>On Wed, Jun 17, 2020 at 11:45 AM Jark Wu <[hidden email]> wrote:
>>
>> Hi everyone,
>>
>> Before going to use {% link %} and adding a CI to force using link tag, I
>> suggest adding a CI profile to check broken links first.
>>
>> The background is that, recently, I noticed that many contributors are
>> beginning to use link tags, but forget to link ".zh.md" instead of ".md" in
>> Chinese documentation.
>> This leads to the docs build failing in the last two days [1]. I have fixed
>> a couple of broken links. But if we don't have a CI mechanism, this would
>> make the docs build unstable.
>>
>> Best,
>> Jark
>>
>> [1]: https://ci.apache.org/builders/flink-docs-master
>>
>> On Mon, 15 Jun 2020 at 12:48, Congxian Qiu <[hidden email]> wrote:
>>
>> > +1 to use {% link %} tag and add a check during CI.
>> > for Chinese doc, will suggest the Chinese translate contributor use the {%
>> > link %} tag when reviewing the translate pr.
>> >
>> > Best,
>> > Congxian
>> >
>> >
>> > Jark Wu <[hidden email]> 于2020年6月10日周三上午10:48写道：
>> >
>> > > +1 to use {% link %} tag and add check in CI.
>> > >
>> > > Tips: if want to link a Chinese page, should write: [CLI]({% link ops/
>> > > cli.zh.md %})
>> > >
>> > > Best,
>> > > Jark
>> > >
>> > > On Wed, 10 Jun 2020 at 10:30, Yangze Guo <[hidden email]> wrote:
>> > >
>> > > > Thanks for that reminder, Seth!
>> > > >
>> > > > +1 to add a check during CI if possible.
>> > > >
>> > > > Best,
>> > > > Yangze Guo
>> > > >
>> > > > On Wed, Jun 10, 2020 at 3:04 AM Kostas Kloudas <[hidden email]>
>> > > wrote:
>> > > > >
>> > > > > Thanks for the heads up Seth!
>> > > > >
>> > > > > Kostas
>> > > > >
>> > > > > On Tue, Jun 9, 2020 at 7:27 PM Seth Wiesman <[hidden email]>
>> > > wrote:
>> > > > > >
>> > > > > > The tag is new to Jekyll 4.0 which we only recently updated to.
>> > > > > >
>> > > > > > There are a lot of existing tags that would need to be updated
>> > first
>> > > :)
>> > > > > > I opened a ticket to track that work and then yes that would make
>> > > > sense.
>> > > > > >
>> > > > > > Seth
>> > > > > >
>> > > > > > [1] https://issues.apache.org/jira/browse/FLINK-18193
>> > > > > >
>> > > > > > On Tue, Jun 9, 2020 at 12:05 PM Robert Metzger <
>> > [hidden email]>
>> > > > wrote:
>> > > > > >
>> > > > > > > Thanks for the reminder. I was also not aware of this tag!
>> > > > > > >
>> > > > > > > How about enforcing the use of this tag through CI?
>> > > > > > > We could for example grep through the added lines of all changes
>> > in
>> > > > docs/
>> > > > > > > and fail the build if we see the wrong pattern.
>> > > > > > >
>> > > > > > >
>> > > > > > > On Tue, Jun 9, 2020 at 4:37 PM Seth Wiesman <[hidden email]
>> > >
>> > > > wrote:
>> > > > > > >
>> > > > > > > > Whoops, responded from the wrong email :)
>> > > > > > > >
>> > > > > > > > Thank you for noticing that, the guide is out of date. I will
>> > fix
>> > > > that
>> > > > > > > > immediately!
>> > > > > > > >
>> > > > > > > > On Tue, Jun 9, 2020 at 9:36 AM Seth Wiesman <
>> > [hidden email]>
>> > > > wrote:
>> > > > > > > >
>> > > > > > > > > Thank you for noticing that, the guide is out of date. I will
>> > > > fix that
>> > > > > > > > > immediately!
>> > > > > > > > >
>> > > > > > > > > On Tue, Jun 9, 2020 at 9:34 AM Dawid Wysakowicz <
>> > > > > > > [hidden email]>
>> > > > > > > > > wrote:
>> > > > > > > > >
>> > > > > > > > >> Hi Seth,
>> > > > > > > > >>
>> > > > > > > > >> Thanks I did not know that.
>> > > > > > > > >>
>> > > > > > > > >> I am not entirely sure, but I think our documentation guide
>> > is
>> > > > > > > slightly
>> > > > > > > > >> outdated on that manner (
>> > > > > > > > >> https://flink.apache.org/contributing/docs-style.html) Or
>> > is
>> > > > there a
>> > > > > > > > >> mistake in your example? Our guide recommends:
>> > > > > > > > >>
>> > > > > > > > >> [CLI]({{ site.baseurl }}{% link ops/cli.md %})
>> > > > > > > > >>
>> > > > > > > > >>
>> > > > > > > > >> Best,
>> > > > > > > > >>
>> > > > > > > > >> Dawid
>> > > > > > > > >> On 09/06/2020 16:20, Seth Wiesman wrote:
>> > > > > > > > >>
>> > > > > > > > >> Hi Everyone!
>> > > > > > > > >>
>> > > > > > > > >> As we are seeing an influx of documentation PRs in
>> > > anticipation
>> > > > of the
>> > > > > > > > 1.11
>> > > > > > > > >> release I would like to remind everyone to use the {% link
>> > %}
>> > > > tag when
>> > > > > > > > >> cross-linking pages[1]. This is opposed to creating a link
>> > > > based on
>> > > > > > > > >> site.baseurl.
>> > > > > > > > >>
>> > > > > > > > >> Going forward a link such as:
>> > > > > > > > >>
>> > > > > > > > >> [CLI]({% site.baseurl %}/ops/cli.html)
>> > > > > > > > >>
>> > > > > > > > >> Should be written as:
>> > > > > > > > >>
>> > > > > > > > >> [CLI]({% link ops/cli.md %})
>> > > > > > > > >>
>> > > > > > > > >> This tag will fail the build on broken links which will help
>> > > us
>> > > > > > > prevent
>> > > > > > > > >> 404s on the website.
>> > > > > > > > >>
>> > > > > > > > >> You can see a good example of the link tag in action
>> > here[2].
>> > > > > > > > >>
>> > > > > > > > >> Seth
>> > > > > > > > >>
>> > > > > > > > >> [1] https://jekyllrb.com/docs/liquid/tags/
>> > > > > > > > >> [2]
>> > > > > > > >
>> > > > > > >
>> > > >
>> > >
>> > https://github.com/apache/flink/blame/b6ea96251d101ca25aa6a6b92170cfa4274b4cc3/docs/index.md#L65-L67
>> > > > > > > > >>
>> > > > > > > > >>
>> > > > > > > > >
>> > > > > > > > > --
>> > > > > > > > >
>> > > > > > > > > Seth Wiesman | Solutions Architect
>> > > > > > > > >
>> > > > > > > > > +1 314 387 1463
>> > > > > > > > >
>> > > > > > > > > <https://www.ververica.com/>
>> > > > > > > > >
>> > > > > > > > > Follow us @VervericaData
>> > > > > > > > >
>> > > > > > > > > --
>> > > > > > > > >
>> > > > > > > > > Join Flink Forward <https://flink-forward.org/> - The Apache
>> > > > Flink
>> > > > > > > > > Conference
>> > > > > > > > >
>> > > > > > > > > Stream Processing | Event Driven | Real Time
>> > > > > > > > >
>> > > > > > > >
>> > > > > > >
>> > > >
>> > >
>> >

Jark Wu-2

Re: Re: [Reminder] Prefer {% link %} tag in documentation

Yes, I think this rule also applies to resources, e.g. "{% link
fig/stream_barriers.svg %}". Because it will help us to prevent dead links.
Please correct me if I'm wrong @Seth.

Best,
Jark

On Fri, 26 Jun 2020 at 22:30, Roc Marshal <[hidden email]> wrote:

> Hi everyone,
>
>
> There is a question about optimizing and translating documents. If the
> link target is an internal image, does the link method need to be set as
> the same way that link target is other pages.
> Such as : Should """ src="{{ site.baseurl }}/fig/stream_barriers.svg"
> """ be written as """ src="{% link fig/stream_barriers.svg %} """ ?
>
>
> Thank you.
> Best Roc.
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
> At 2020-06-17 12:38:38, "Yangze Guo" <[hidden email]> wrote:
> >I second Jark's comment. We need a CI mechanism to ensure this.
> >
> >Best,
> >Yangze Guo
> >
> >On Wed, Jun 17, 2020 at 11:45 AM Jark Wu <[hidden email]> wrote:
> >>
> >> Hi everyone,
> >>
> >> Before going to use {% link %} and adding a CI to force using link tag,
> I
> >> suggest adding a CI profile to check broken links first.
> >>
> >> The background is that, recently, I noticed that many contributors are
> >> beginning to use link tags, but forget to link ".zh.md" instead of
> ".md" in
> >> Chinese documentation.
> >> This leads to the docs build failing in the last two days [1]. I have
> fixed
> >> a couple of broken links. But if we don't have a CI mechanism, this
> would
> >> make the docs build unstable.
> >>
> >> Best,
> >> Jark
> >>
> >> [1]: https://ci.apache.org/builders/flink-docs-master
> >>
> >> On Mon, 15 Jun 2020 at 12:48, Congxian Qiu <[hidden email]>
> wrote:
> >>
> >> > +1 to use {% link %} tag and add a check during CI.
> >> > for Chinese doc, will suggest the Chinese translate contributor use
> the {%
> >> > link %} tag when reviewing the translate pr.
> >> >
> >> > Best,
> >> > Congxian
> >> >
> >> >
> >> > Jark Wu <[hidden email]> 于2020年6月10日周三上午10:48写道：
> >> >
> >> > > +1 to use {% link %} tag and add check in CI.
> >> > >
> >> > > Tips: if want to link a Chinese page, should write: [CLI]({% link
> ops/
> >> > > cli.zh.md %})
> >> > >
> >> > > Best,
> >> > > Jark
> >> > >
> >> > > On Wed, 10 Jun 2020 at 10:30, Yangze Guo <[hidden email]>
> wrote:
> >> > >
> >> > > > Thanks for that reminder, Seth!
> >> > > >
> >> > > > +1 to add a check during CI if possible.
> >> > > >
> >> > > > Best,
> >> > > > Yangze Guo
> >> > > >
> >> > > > On Wed, Jun 10, 2020 at 3:04 AM Kostas Kloudas <
> [hidden email]>
> >> > > wrote:
> >> > > > >
> >> > > > > Thanks for the heads up Seth!
> >> > > > >
> >> > > > > Kostas
> >> > > > >
> >> > > > > On Tue, Jun 9, 2020 at 7:27 PM Seth Wiesman <
> [hidden email]>
> >> > > wrote:
> >> > > > > >
> >> > > > > > The tag is new to Jekyll 4.0 which we only recently updated
> to.
> >> > > > > >
> >> > > > > > There are a lot of existing tags that would need to be updated
> >> > first
> >> > > :)
> >> > > > > > I opened a ticket to track that work and then yes that would
> make
> >> > > > sense.
> >> > > > > >
> >> > > > > > Seth
> >> > > > > >
> >> > > > > > [1] https://issues.apache.org/jira/browse/FLINK-18193
> >> > > > > >
> >> > > > > > On Tue, Jun 9, 2020 at 12:05 PM Robert Metzger <
> >> > [hidden email]>
> >> > > > wrote:
> >> > > > > >
> >> > > > > > > Thanks for the reminder. I was also not aware of this tag!
> >> > > > > > >
> >> > > > > > > How about enforcing the use of this tag through CI?
> >> > > > > > > We could for example grep through the added lines of all
> changes
> >> > in
> >> > > > docs/
> >> > > > > > > and fail the build if we see the wrong pattern.
> >> > > > > > >
> >> > > > > > >
> >> > > > > > > On Tue, Jun 9, 2020 at 4:37 PM Seth Wiesman <
> [hidden email]
> >> > >
> >> > > > wrote:
> >> > > > > > >
> >> > > > > > > > Whoops, responded from the wrong email :)
> >> > > > > > > >
> >> > > > > > > > Thank you for noticing that, the guide is out of date. I
> will
> >> > fix
> >> > > > that
> >> > > > > > > > immediately!
> >> > > > > > > >
> >> > > > > > > > On Tue, Jun 9, 2020 at 9:36 AM Seth Wiesman <
> >> > [hidden email]>
> >> > > > wrote:
> >> > > > > > > >
> >> > > > > > > > > Thank you for noticing that, the guide is out of date.
> I will
> >> > > > fix that
> >> > > > > > > > > immediately!
> >> > > > > > > > >
> >> > > > > > > > > On Tue, Jun 9, 2020 at 9:34 AM Dawid Wysakowicz <
> >> > > > > > > [hidden email]>
> >> > > > > > > > > wrote:
> >> > > > > > > > >
> >> > > > > > > > >> Hi Seth,
> >> > > > > > > > >>
> >> > > > > > > > >> Thanks I did not know that.
> >> > > > > > > > >>
> >> > > > > > > > >> I am not entirely sure, but I think our documentation
> guide
> >> > is
> >> > > > > > > slightly
> >> > > > > > > > >> outdated on that manner (
> >> > > > > > > > >> https://flink.apache.org/contributing/docs-style.html)
> Or
> >> > is
> >> > > > there a
> >> > > > > > > > >> mistake in your example? Our guide recommends:
> >> > > > > > > > >>
> >> > > > > > > > >> [CLI]({{ site.baseurl }}{% link ops/cli.md %})
> >> > > > > > > > >>
> >> > > > > > > > >>
> >> > > > > > > > >> Best,
> >> > > > > > > > >>
> >> > > > > > > > >> Dawid
> >> > > > > > > > >> On 09/06/2020 16:20, Seth Wiesman wrote:
> >> > > > > > > > >>
> >> > > > > > > > >> Hi Everyone!
> >> > > > > > > > >>
> >> > > > > > > > >> As we are seeing an influx of documentation PRs in
> >> > > anticipation
> >> > > > of the
> >> > > > > > > > 1.11
> >> > > > > > > > >> release I would like to remind everyone to use the {%
> link
> >> > %}
> >> > > > tag when
> >> > > > > > > > >> cross-linking pages[1]. This is opposed to creating a
> link
> >> > > > based on
> >> > > > > > > > >> site.baseurl.
> >> > > > > > > > >>
> >> > > > > > > > >> Going forward a link such as:
> >> > > > > > > > >>
> >> > > > > > > > >> [CLI]({% site.baseurl %}/ops/cli.html)
> >> > > > > > > > >>
> >> > > > > > > > >> Should be written as:
> >> > > > > > > > >>
> >> > > > > > > > >> [CLI]({% link ops/cli.md %})
> >> > > > > > > > >>
> >> > > > > > > > >> This tag will fail the build on broken links which
> will help
> >> > > us
> >> > > > > > > prevent
> >> > > > > > > > >> 404s on the website.
> >> > > > > > > > >>
> >> > > > > > > > >> You can see a good example of the link tag in action
> >> > here[2].
> >> > > > > > > > >>
> >> > > > > > > > >> Seth
> >> > > > > > > > >>
> >> > > > > > > > >> [1] https://jekyllrb.com/docs/liquid/tags/
> >> > > > > > > > >> [2]
> >> > > > > > > >
> >> > > > > > >
> >> > > >
> >> > >
> >> >
> https://github.com/apache/flink/blame/b6ea96251d101ca25aa6a6b92170cfa4274b4cc3/docs/index.md#L65-L67
> >> > > > > > > > >>
> >> > > > > > > > >>
> >> > > > > > > > >
> >> > > > > > > > > --
> >> > > > > > > > >
> >> > > > > > > > > Seth Wiesman | Solutions Architect
> >> > > > > > > > >
> >> > > > > > > > > +1 314 387 1463
> >> > > > > > > > >
> >> > > > > > > > > <https://www.ververica.com/>
> >> > > > > > > > >
> >> > > > > > > > > Follow us @VervericaData
> >> > > > > > > > >
> >> > > > > > > > > --
> >> > > > > > > > >
> >> > > > > > > > > Join Flink Forward <https://flink-forward.org/> - The
> Apache
> >> > > > Flink
> >> > > > > > > > > Conference
> >> > > > > > > > >
> >> > > > > > > > > Stream Processing | Event Driven | Real Time
> >> > > > > > > > >
> >> > > > > > > >
> >> > > > > > >
> >> > > >
> >> > >
> >> >
>

David Anderson-3

Re: Re: [Reminder] Prefer {% link %} tag in documentation

Yes, it makes sense to rewrite img tags and other resources using {% link
... %}, but, for example,

<img src="{{ site.baseurl }}/fig/stream_barriers.svg" ...

needs to become

<img src="{% link /fig/stream_barriers.svg %}" ...

In other words, use absolute paths to the figures, rather than relative
ones.

I just tested this, and it is helpful in finding dead/broken links.

Best,
David

On Sun, Jun 28, 2020 at 5:43 AM Jark Wu <[hidden email]> wrote:

> Yes, I think this rule also applies to resources, e.g. "{% link
> fig/stream_barriers.svg %}". Because it will help us to prevent dead links.
> Please correct me if I'm wrong @Seth.
>
> Best,
> Jark
>
> On Fri, 26 Jun 2020 at 22:30, Roc Marshal <[hidden email]> wrote:
>
> > Hi everyone,
> >
> >
> > There is a question about optimizing and translating documents. If the
> > link target is an internal image, does the link method need to be set as
> > the same way that link target is other pages.
> > Such as : Should """ src="{{ site.baseurl }}/fig/stream_barriers.svg"
> > """ be written as """ src="{% link fig/stream_barriers.svg %} """ ?
> >
> >
> > Thank you.
> > Best Roc.
> >
> >
> >
> >
> >
> >
> >
> >
> >
> >
> >
> >
> >
> >
> >
> >
> > At 2020-06-17 12:38:38, "Yangze Guo" <[hidden email]> wrote:
> > >I second Jark's comment. We need a CI mechanism to ensure this.
> > >
> > >Best,
> > >Yangze Guo
> > >
> > >On Wed, Jun 17, 2020 at 11:45 AM Jark Wu <[hidden email]> wrote:
> > >>
> > >> Hi everyone,
> > >>
> > >> Before going to use {% link %} and adding a CI to force using link
> tag,
> > I
> > >> suggest adding a CI profile to check broken links first.
> > >>
> > >> The background is that, recently, I noticed that many contributors are
> > >> beginning to use link tags, but forget to link ".zh.md" instead of
> > ".md" in
> > >> Chinese documentation.
> > >> This leads to the docs build failing in the last two days [1]. I have
> > fixed
> > >> a couple of broken links. But if we don't have a CI mechanism, this
> > would
> > >> make the docs build unstable.
> > >>
> > >> Best,
> > >> Jark
> > >>
> > >> [1]: https://ci.apache.org/builders/flink-docs-master
> > >>
> > >> On Mon, 15 Jun 2020 at 12:48, Congxian Qiu <[hidden email]>
> > wrote:
> > >>
> > >> > +1 to use {% link %} tag and add a check during CI.
> > >> > for Chinese doc, will suggest the Chinese translate contributor use
> > the {%
> > >> > link %} tag when reviewing the translate pr.
> > >> >
> > >> > Best,
> > >> > Congxian
> > >> >
> > >> >
> > >> > Jark Wu <[hidden email]> 于2020年6月10日周三上午10:48写道：
> > >> >
> > >> > > +1 to use {% link %} tag and add check in CI.
> > >> > >
> > >> > > Tips: if want to link a Chinese page, should write: [CLI]({% link
> > ops/
> > >> > > cli.zh.md %})
> > >> > >
> > >> > > Best,
> > >> > > Jark
> > >> > >
> > >> > > On Wed, 10 Jun 2020 at 10:30, Yangze Guo <[hidden email]>
> > wrote:
> > >> > >
> > >> > > > Thanks for that reminder, Seth!
> > >> > > >
> > >> > > > +1 to add a check during CI if possible.
> > >> > > >
> > >> > > > Best,
> > >> > > > Yangze Guo
> > >> > > >
> > >> > > > On Wed, Jun 10, 2020 at 3:04 AM Kostas Kloudas <
> > [hidden email]>
> > >> > > wrote:
> > >> > > > >
> > >> > > > > Thanks for the heads up Seth!
> > >> > > > >
> > >> > > > > Kostas
> > >> > > > >
> > >> > > > > On Tue, Jun 9, 2020 at 7:27 PM Seth Wiesman <
> > [hidden email]>
> > >> > > wrote:
> > >> > > > > >
> > >> > > > > > The tag is new to Jekyll 4.0 which we only recently updated
> > to.
> > >> > > > > >
> > >> > > > > > There are a lot of existing tags that would need to be
> updated
> > >> > first
> > >> > > :)
> > >> > > > > > I opened a ticket to track that work and then yes that would
> > make
> > >> > > > sense.
> > >> > > > > >
> > >> > > > > > Seth
> > >> > > > > >
> > >> > > > > > [1] https://issues.apache.org/jira/browse/FLINK-18193
> > >> > > > > >
> > >> > > > > > On Tue, Jun 9, 2020 at 12:05 PM Robert Metzger <
> > >> > [hidden email]>
> > >> > > > wrote:
> > >> > > > > >
> > >> > > > > > > Thanks for the reminder. I was also not aware of this tag!
> > >> > > > > > >
> > >> > > > > > > How about enforcing the use of this tag through CI?
> > >> > > > > > > We could for example grep through the added lines of all
> > changes
> > >> > in
> > >> > > > docs/
> > >> > > > > > > and fail the build if we see the wrong pattern.
> > >> > > > > > >
> > >> > > > > > >
> > >> > > > > > > On Tue, Jun 9, 2020 at 4:37 PM Seth Wiesman <
> > [hidden email]
> > >> > >
> > >> > > > wrote:
> > >> > > > > > >
> > >> > > > > > > > Whoops, responded from the wrong email :)
> > >> > > > > > > >
> > >> > > > > > > > Thank you for noticing that, the guide is out of date. I
> > will
> > >> > fix
> > >> > > > that
> > >> > > > > > > > immediately!
> > >> > > > > > > >
> > >> > > > > > > > On Tue, Jun 9, 2020 at 9:36 AM Seth Wiesman <
> > >> > [hidden email]>
> > >> > > > wrote:
> > >> > > > > > > >
> > >> > > > > > > > > Thank you for noticing that, the guide is out of date.
> > I will
> > >> > > > fix that
> > >> > > > > > > > > immediately!
> > >> > > > > > > > >
> > >> > > > > > > > > On Tue, Jun 9, 2020 at 9:34 AM Dawid Wysakowicz <
> > >> > > > > > > [hidden email]>
> > >> > > > > > > > > wrote:
> > >> > > > > > > > >
> > >> > > > > > > > >> Hi Seth,
> > >> > > > > > > > >>
> > >> > > > > > > > >> Thanks I did not know that.
> > >> > > > > > > > >>
> > >> > > > > > > > >> I am not entirely sure, but I think our documentation
> > guide
> > >> > is
> > >> > > > > > > slightly
> > >> > > > > > > > >> outdated on that manner (
> > >> > > > > > > > >>
> https://flink.apache.org/contributing/docs-style.html)
> > Or
> > >> > is
> > >> > > > there a
> > >> > > > > > > > >> mistake in your example? Our guide recommends:
> > >> > > > > > > > >>
> > >> > > > > > > > >> [CLI]({{ site.baseurl }}{% link ops/cli.md %})
> > >> > > > > > > > >>
> > >> > > > > > > > >>
> > >> > > > > > > > >> Best,
> > >> > > > > > > > >>
> > >> > > > > > > > >> Dawid
> > >> > > > > > > > >> On 09/06/2020 16:20, Seth Wiesman wrote:
> > >> > > > > > > > >>
> > >> > > > > > > > >> Hi Everyone!
> > >> > > > > > > > >>
> > >> > > > > > > > >> As we are seeing an influx of documentation PRs in
> > >> > > anticipation
> > >> > > > of the
> > >> > > > > > > > 1.11
> > >> > > > > > > > >> release I would like to remind everyone to use the {%
> > link
> > >> > %}
> > >> > > > tag when
> > >> > > > > > > > >> cross-linking pages[1]. This is opposed to creating a
> > link
> > >> > > > based on
> > >> > > > > > > > >> site.baseurl.
> > >> > > > > > > > >>
> > >> > > > > > > > >> Going forward a link such as:
> > >> > > > > > > > >>
> > >> > > > > > > > >> [CLI]({% site.baseurl %}/ops/cli.html)
> > >> > > > > > > > >>
> > >> > > > > > > > >> Should be written as:
> > >> > > > > > > > >>
> > >> > > > > > > > >> [CLI]({% link ops/cli.md %})
> > >> > > > > > > > >>
> > >> > > > > > > > >> This tag will fail the build on broken links which
> > will help
> > >> > > us
> > >> > > > > > > prevent
> > >> > > > > > > > >> 404s on the website.
> > >> > > > > > > > >>
> > >> > > > > > > > >> You can see a good example of the link tag in action
> > >> > here[2].
> > >> > > > > > > > >>
> > >> > > > > > > > >> Seth
> > >> > > > > > > > >>
> > >> > > > > > > > >> [1] https://jekyllrb.com/docs/liquid/tags/
> > >> > > > > > > > >> [2]
> > >> > > > > > > >
> > >> > > > > > >
> > >> > > >
> > >> > >
> > >> >
> >
> https://github.com/apache/flink/blame/b6ea96251d101ca25aa6a6b92170cfa4274b4cc3/docs/index.md#L65-L67
> > >> > > > > > > > >>
> > >> > > > > > > > >>
> > >> > > > > > > > >
> > >> > > > > > > > > --
> > >> > > > > > > > >
> > >> > > > > > > > > Seth Wiesman | Solutions Architect
> > >> > > > > > > > >
> > >> > > > > > > > > +1 314 387 1463
> > >> > > > > > > > >
> > >> > > > > > > > > <https://www.ververica.com/>
> > >> > > > > > > > >
> > >> > > > > > > > > Follow us @VervericaData
> > >> > > > > > > > >
> > >> > > > > > > > > --
> > >> > > > > > > > >
> > >> > > > > > > > > Join Flink Forward <https://flink-forward.org/> - The
> > Apache
> > >> > > > Flink
> > >> > > > > > > > > Conference
> > >> > > > > > > > >
> > >> > > > > > > > > Stream Processing | Event Driven | Real Time
> > >> > > > > > > > >
> > >> > > > > > > >
> > >> > > > > > >
> > >> > > >
> > >> > >
> > >> >
> >
>

RocMarshal

Re: [Reminder] Prefer {% link %} tag in documentation

Hi, David. I am sorry for late reply.
I will check pages which were translated by me according to your reply.

Thank you so much.
Best,
Roc.

| |
Roc Marshal
|
|
邮箱：[hidden email]
|

签名由网易邮箱大师定制

On 06/30/2020 15:20, David Anderson wrote:
Yes, it makes sense to rewrite img tags and other resources using {% link ... %}, but, for example,

<img src="{{ site.baseurl }}/fig/stream_barriers.svg" ...
needs to become

<img src="{% link /fig/stream_barriers.svg %}" ...
In other words, use absolute paths to the figures, rather than relative ones.

I just tested this, and it is helpful in finding dead/broken links.

Best,
David

On Sun, Jun 28, 2020 at 5:43 AM Jark Wu <[hidden email]> wrote:

Yes, I think this rule also applies to resources, e.g. "{% link
fig/stream_barriers.svg %}". Because it will help us to prevent dead links.
Please correct me if I'm wrong @Seth.

Best,
Jark

On Fri, 26 Jun 2020 at 22:30, Roc Marshal <[hidden email]> wrote: