Re: [DISCUSS] Asciidoc Header Reformat
Looks good! Robert Dale On Tue, Sep 26, 2017 at 1:38 PM, Daniel Kuppitzwrote: > Yea, that was the right regex to look for these errors: grep '^== $' > $(find . -name "*.asciidoc") > Got them all now, will push the fix in a few. > > Cheers, > Daniel > > > On Tue, Sep 26, 2017 at 10:30 AM, Robert Dale wrote: > > > docs/src/reference/gremlin-applications.asciidoc: > > @@ -294,8 +288,7 @@ v[2] > > v[3] > > v[4] > > v[5] > > -v[6] > > - > > +== v[6] > > > > It is also possible to pass arguments to scripts. Any parameters > > following the file name specification are treated > > as arguments to the script. They are collected into a list and passed in > > as a variable called "args". The following > > @@ -316,8 +309,7 @@ When executed from the command line a parameter can > be > > supplied: > > $ bin/gremlin.sh -e gremlin.groovy marko > > v[1] > > $ bin/gremlin.sh -e gremlin.groovy vadas > > -v[2] > > - > > +== v[2] > > > > > > > > Robert Dale > > > > On Tue, Sep 26, 2017 at 1:22 PM, Robert Dale wrote: > > > > > Looks like master CHANGELOG picked up a conflict... > > > > > > <<< HEAD > > > == TinkerPop 3.3.0 (Gremlin Symphony #40 in G Minor) > > > . > > > . > > > . > > > > > > == TinkerPop 3.2.0 (Nine Inch Gremlins) > > > === > > > == TinkerPop 3.2.0 (Nine Inch Gremlins) > > > >>> asciidoc-header > > > > > > > > > Robert Dale > > > > > > On Tue, Sep 26, 2017 at 1:19 PM, Daniel Kuppitz > wrote: > > > > > >> Ah, good find! That's the same issue I've had in the reference docs > and > > >> recipes. Fixed. > > >> > > >> Cheers, > > >> Daniel > > >> > > >> > > >> On Tue, Sep 26, 2017 at 10:08 AM, Robert Dale > > wrote: > > >> > > >> > git grep "== ==" > > >> > > > >> > Robert Dale > > >> > > > >> > On Tue, Sep 26, 2017 at 1:07 PM, Robert Dale > > wrote: > > >> > > > >> > > I see things like '== ==>6' in docs/src/upgrade/release-3.2. > > >> > > x-incubating.html > > >> > > > > >> > > Robert Dale > > >> > > > > >> > > On Tue, Sep 26, 2017 at 12:57 PM, Robert Dale > > >> wrote: > > >> > > > > >> > >> ^d > > >> > >> > > >> > >> Robert Dale > > >> > >> > > >> > >> On Tue, Sep 26, 2017 at 12:56 PM, Daniel Kuppitz > > > >> > wrote: > > >> > >> > > >> > >>> This is one. I pushed the changes to tp32/ and master/. I only > > found > > >> > (and > > >> > >>> fixed) 2 minor issues, where my script produced a little > > formatting > > >> > mess. > > >> > >>> > > >> > >>> Cheers, > > >> > >>> Daniel > > >> > >>> > > >> > >>> > > >> > >>> On Tue, Sep 19, 2017 at 9:23 AM, Stephen Mallette < > > >> > spmalle...@gmail.com> > > >> > >>> wrote: > > >> > >>> > > >> > >>> > For whatever reason we started writing our headers in our docs > > >> with: > > >> > >>> > > > >> > >>> > Header > > >> > >>> > == > > >> > >>> > > > >> > >>> > Subheader > > >> > >>> > - > > >> > >>> > > > >> > >>> > SubSubHeader > > >> > >>> > > > >> > >>> > > > >> > >>> > as opposed to the alternative: > > >> > >>> > > > >> > >>> > = This Is A Header > > >> > >>> > > > >> > >>> > == Subheader > > >> > >>> > > > >> > >>> > === SubSubHeader > > >> > >>> > > > >> > >>> > The downside with the first approach is that it forces us to > do > > >> two > > >> > >>> things: > > >> > >>> > > > >> > >>> > 1. Remember the syntax for each header layer (which we all > > >> probably > > >> > >>> know > > >> > >>> > pretty well at this point, but still...) > > >> > >>> > 2. Ensure that the width of the header formatting line is > equal > > to > > >> > the > > >> > >>> > length of the text above it. > > >> > >>> > > > >> > >>> > That second point is a killer sometimes. While I've trained > > >> myself to > > >> > >>> be > > >> > >>> > good at noticing that problem I know it's bitten us in > releases > > >> over > > >> > >>> and > > >> > >>> > over and over again. > > >> > >>> > > > >> > >>> > Anyway, I think going to the alternative formatting would be > > >> better > > >> > and > > >> > >>> > propose to do a full reformatting. Please let me know if there > > are > > >> > any > > >> > >>> > concerns about doing that - if I don't hear back in the next > few > > >> days > > >> > >>> I'll > > >> > >>> > proceed with the changes. > > >> > >>> > > > >> > >>> > Thanks, > > >> > >>> > > > >> > >>> > Stephen > > >> > >>> > > > >> > >>> > > >> > >> > > >> > >> > > >> > > > > >> > > > >> > > > > > > > > >
Re: [DISCUSS] Asciidoc Header Reformat
Yea, that was the right regex to look for these errors: grep '^== $' $(find . -name "*.asciidoc") Got them all now, will push the fix in a few. Cheers, Daniel On Tue, Sep 26, 2017 at 10:30 AM, Robert Dalewrote: > docs/src/reference/gremlin-applications.asciidoc: > @@ -294,8 +288,7 @@ v[2] > v[3] > v[4] > v[5] > -v[6] > - > +== v[6] > > It is also possible to pass arguments to scripts. Any parameters > following the file name specification are treated > as arguments to the script. They are collected into a list and passed in > as a variable called "args". The following > @@ -316,8 +309,7 @@ When executed from the command line a parameter can be > supplied: > $ bin/gremlin.sh -e gremlin.groovy marko > v[1] > $ bin/gremlin.sh -e gremlin.groovy vadas > -v[2] > - > +== v[2] > > > > Robert Dale > > On Tue, Sep 26, 2017 at 1:22 PM, Robert Dale wrote: > > > Looks like master CHANGELOG picked up a conflict... > > > > <<< HEAD > > == TinkerPop 3.3.0 (Gremlin Symphony #40 in G Minor) > > . > > . > > . > > > > == TinkerPop 3.2.0 (Nine Inch Gremlins) > > === > > == TinkerPop 3.2.0 (Nine Inch Gremlins) > > >>> asciidoc-header > > > > > > Robert Dale > > > > On Tue, Sep 26, 2017 at 1:19 PM, Daniel Kuppitz wrote: > > > >> Ah, good find! That's the same issue I've had in the reference docs and > >> recipes. Fixed. > >> > >> Cheers, > >> Daniel > >> > >> > >> On Tue, Sep 26, 2017 at 10:08 AM, Robert Dale > wrote: > >> > >> > git grep "== ==" > >> > > >> > Robert Dale > >> > > >> > On Tue, Sep 26, 2017 at 1:07 PM, Robert Dale > wrote: > >> > > >> > > I see things like '== ==>6' in docs/src/upgrade/release-3.2. > >> > > x-incubating.html > >> > > > >> > > Robert Dale > >> > > > >> > > On Tue, Sep 26, 2017 at 12:57 PM, Robert Dale > >> wrote: > >> > > > >> > >> ^d > >> > >> > >> > >> Robert Dale > >> > >> > >> > >> On Tue, Sep 26, 2017 at 12:56 PM, Daniel Kuppitz > >> > wrote: > >> > >> > >> > >>> This is one. I pushed the changes to tp32/ and master/. I only > found > >> > (and > >> > >>> fixed) 2 minor issues, where my script produced a little > formatting > >> > mess. > >> > >>> > >> > >>> Cheers, > >> > >>> Daniel > >> > >>> > >> > >>> > >> > >>> On Tue, Sep 19, 2017 at 9:23 AM, Stephen Mallette < > >> > spmalle...@gmail.com> > >> > >>> wrote: > >> > >>> > >> > >>> > For whatever reason we started writing our headers in our docs > >> with: > >> > >>> > > >> > >>> > Header > >> > >>> > == > >> > >>> > > >> > >>> > Subheader > >> > >>> > - > >> > >>> > > >> > >>> > SubSubHeader > >> > >>> > > >> > >>> > > >> > >>> > as opposed to the alternative: > >> > >>> > > >> > >>> > = This Is A Header > >> > >>> > > >> > >>> > == Subheader > >> > >>> > > >> > >>> > === SubSubHeader > >> > >>> > > >> > >>> > The downside with the first approach is that it forces us to do > >> two > >> > >>> things: > >> > >>> > > >> > >>> > 1. Remember the syntax for each header layer (which we all > >> probably > >> > >>> know > >> > >>> > pretty well at this point, but still...) > >> > >>> > 2. Ensure that the width of the header formatting line is equal > to > >> > the > >> > >>> > length of the text above it. > >> > >>> > > >> > >>> > That second point is a killer sometimes. While I've trained > >> myself to > >> > >>> be > >> > >>> > good at noticing that problem I know it's bitten us in releases > >> over > >> > >>> and > >> > >>> > over and over again. > >> > >>> > > >> > >>> > Anyway, I think going to the alternative formatting would be > >> better > >> > and > >> > >>> > propose to do a full reformatting. Please let me know if there > are > >> > any > >> > >>> > concerns about doing that - if I don't hear back in the next few > >> days > >> > >>> I'll > >> > >>> > proceed with the changes. > >> > >>> > > >> > >>> > Thanks, > >> > >>> > > >> > >>> > Stephen > >> > >>> > > >> > >>> > >> > >> > >> > >> > >> > > > >> > > >> > > > > >
Re: [DISCUSS] Asciidoc Header Reformat
That was master. tp32 is: @@ -278,8 +272,7 @@ v[2] v[3] v[4] v[5] -v[6] - +== v[6] It is also possible to pass arguments to scripts. Any parameters following the file name specification are treated as arguments to the script. They are collected into a list and passed in as a variable called "args". The following @@ -300,8 +293,7 @@ When executed from the command line a parameter can be supplied: $ bin/gremlin.sh -e gremlin.groovy marko v[1] $ bin/gremlin.sh -e gremlin.groovy vadas -v[2] - +== v[2] Robert Dale On Tue, Sep 26, 2017 at 1:30 PM, Robert Dalewrote: > docs/src/reference/gremlin-applications.asciidoc: > @@ -294,8 +288,7 @@ v[2] > v[3] > v[4] > v[5] > -v[6] > - > +== v[6] > > It is also possible to pass arguments to scripts. Any parameters > following the file name specification are treated > as arguments to the script. They are collected into a list and passed in > as a variable called "args". The following > @@ -316,8 +309,7 @@ When executed from the command line a parameter can be > supplied: > $ bin/gremlin.sh -e gremlin.groovy marko > v[1] > $ bin/gremlin.sh -e gremlin.groovy vadas > -v[2] > - > +== v[2] > > > > Robert Dale > > On Tue, Sep 26, 2017 at 1:22 PM, Robert Dale wrote: > >> Looks like master CHANGELOG picked up a conflict... >> >> <<< HEAD >> == TinkerPop 3.3.0 (Gremlin Symphony #40 in G Minor) >> . >> . >> . >> >> == TinkerPop 3.2.0 (Nine Inch Gremlins) >> === >> == TinkerPop 3.2.0 (Nine Inch Gremlins) >> >>> asciidoc-header >> >> >> Robert Dale >> >> On Tue, Sep 26, 2017 at 1:19 PM, Daniel Kuppitz wrote: >> >>> Ah, good find! That's the same issue I've had in the reference docs and >>> recipes. Fixed. >>> >>> Cheers, >>> Daniel >>> >>> >>> On Tue, Sep 26, 2017 at 10:08 AM, Robert Dale wrote: >>> >>> > git grep "== ==" >>> > >>> > Robert Dale >>> > >>> > On Tue, Sep 26, 2017 at 1:07 PM, Robert Dale >>> wrote: >>> > >>> > > I see things like '== ==>6' in docs/src/upgrade/release-3.2. >>> > > x-incubating.html >>> > > >>> > > Robert Dale >>> > > >>> > > On Tue, Sep 26, 2017 at 12:57 PM, Robert Dale >>> wrote: >>> > > >>> > >> ^d >>> > >> >>> > >> Robert Dale >>> > >> >>> > >> On Tue, Sep 26, 2017 at 12:56 PM, Daniel Kuppitz >>> > wrote: >>> > >> >>> > >>> This is one. I pushed the changes to tp32/ and master/. I only >>> found >>> > (and >>> > >>> fixed) 2 minor issues, where my script produced a little formatting >>> > mess. >>> > >>> >>> > >>> Cheers, >>> > >>> Daniel >>> > >>> >>> > >>> >>> > >>> On Tue, Sep 19, 2017 at 9:23 AM, Stephen Mallette < >>> > spmalle...@gmail.com> >>> > >>> wrote: >>> > >>> >>> > >>> > For whatever reason we started writing our headers in our docs >>> with: >>> > >>> > >>> > >>> > Header >>> > >>> > == >>> > >>> > >>> > >>> > Subheader >>> > >>> > - >>> > >>> > >>> > >>> > SubSubHeader >>> > >>> > >>> > >>> > >>> > >>> > as opposed to the alternative: >>> > >>> > >>> > >>> > = This Is A Header >>> > >>> > >>> > >>> > == Subheader >>> > >>> > >>> > >>> > === SubSubHeader >>> > >>> > >>> > >>> > The downside with the first approach is that it forces us to do >>> two >>> > >>> things: >>> > >>> > >>> > >>> > 1. Remember the syntax for each header layer (which we all >>> probably >>> > >>> know >>> > >>> > pretty well at this point, but still...) >>> > >>> > 2. Ensure that the width of the header formatting line is equal >>> to >>> > the >>> > >>> > length of the text above it. >>> > >>> > >>> > >>> > That second point is a killer sometimes. While I've trained >>> myself to >>> > >>> be >>> > >>> > good at noticing that problem I know it's bitten us in releases >>> over >>> > >>> and >>> > >>> > over and over again. >>> > >>> > >>> > >>> > Anyway, I think going to the alternative formatting would be >>> better >>> > and >>> > >>> > propose to do a full reformatting. Please let me know if there >>> are >>> > any >>> > >>> > concerns about doing that - if I don't hear back in the next few >>> days >>> > >>> I'll >>> > >>> > proceed with the changes. >>> > >>> > >>> > >>> > Thanks, >>> > >>> > >>> > >>> > Stephen >>> > >>> > >>> > >>> >>> > >> >>> > >> >>> > > >>> > >>> >> >> >
Re: [DISCUSS] Asciidoc Header Reformat
docs/src/reference/gremlin-applications.asciidoc: @@ -294,8 +288,7 @@ v[2] v[3] v[4] v[5] -v[6] - +== v[6] It is also possible to pass arguments to scripts. Any parameters following the file name specification are treated as arguments to the script. They are collected into a list and passed in as a variable called "args". The following @@ -316,8 +309,7 @@ When executed from the command line a parameter can be supplied: $ bin/gremlin.sh -e gremlin.groovy marko v[1] $ bin/gremlin.sh -e gremlin.groovy vadas -v[2] - +== v[2] Robert Dale On Tue, Sep 26, 2017 at 1:22 PM, Robert Dalewrote: > Looks like master CHANGELOG picked up a conflict... > > <<< HEAD > == TinkerPop 3.3.0 (Gremlin Symphony #40 in G Minor) > . > . > . > > == TinkerPop 3.2.0 (Nine Inch Gremlins) > === > == TinkerPop 3.2.0 (Nine Inch Gremlins) > >>> asciidoc-header > > > Robert Dale > > On Tue, Sep 26, 2017 at 1:19 PM, Daniel Kuppitz wrote: > >> Ah, good find! That's the same issue I've had in the reference docs and >> recipes. Fixed. >> >> Cheers, >> Daniel >> >> >> On Tue, Sep 26, 2017 at 10:08 AM, Robert Dale wrote: >> >> > git grep "== ==" >> > >> > Robert Dale >> > >> > On Tue, Sep 26, 2017 at 1:07 PM, Robert Dale wrote: >> > >> > > I see things like '== ==>6' in docs/src/upgrade/release-3.2. >> > > x-incubating.html >> > > >> > > Robert Dale >> > > >> > > On Tue, Sep 26, 2017 at 12:57 PM, Robert Dale >> wrote: >> > > >> > >> ^d >> > >> >> > >> Robert Dale >> > >> >> > >> On Tue, Sep 26, 2017 at 12:56 PM, Daniel Kuppitz >> > wrote: >> > >> >> > >>> This is one. I pushed the changes to tp32/ and master/. I only found >> > (and >> > >>> fixed) 2 minor issues, where my script produced a little formatting >> > mess. >> > >>> >> > >>> Cheers, >> > >>> Daniel >> > >>> >> > >>> >> > >>> On Tue, Sep 19, 2017 at 9:23 AM, Stephen Mallette < >> > spmalle...@gmail.com> >> > >>> wrote: >> > >>> >> > >>> > For whatever reason we started writing our headers in our docs >> with: >> > >>> > >> > >>> > Header >> > >>> > == >> > >>> > >> > >>> > Subheader >> > >>> > - >> > >>> > >> > >>> > SubSubHeader >> > >>> > >> > >>> > >> > >>> > as opposed to the alternative: >> > >>> > >> > >>> > = This Is A Header >> > >>> > >> > >>> > == Subheader >> > >>> > >> > >>> > === SubSubHeader >> > >>> > >> > >>> > The downside with the first approach is that it forces us to do >> two >> > >>> things: >> > >>> > >> > >>> > 1. Remember the syntax for each header layer (which we all >> probably >> > >>> know >> > >>> > pretty well at this point, but still...) >> > >>> > 2. Ensure that the width of the header formatting line is equal to >> > the >> > >>> > length of the text above it. >> > >>> > >> > >>> > That second point is a killer sometimes. While I've trained >> myself to >> > >>> be >> > >>> > good at noticing that problem I know it's bitten us in releases >> over >> > >>> and >> > >>> > over and over again. >> > >>> > >> > >>> > Anyway, I think going to the alternative formatting would be >> better >> > and >> > >>> > propose to do a full reformatting. Please let me know if there are >> > any >> > >>> > concerns about doing that - if I don't hear back in the next few >> days >> > >>> I'll >> > >>> > proceed with the changes. >> > >>> > >> > >>> > Thanks, >> > >>> > >> > >>> > Stephen >> > >>> > >> > >>> >> > >> >> > >> >> > > >> > >> > >
Re: [DISCUSS] Asciidoc Header Reformat
Looks like master CHANGELOG picked up a conflict... <<< HEAD == TinkerPop 3.3.0 (Gremlin Symphony #40 in G Minor) . . . == TinkerPop 3.2.0 (Nine Inch Gremlins) === == TinkerPop 3.2.0 (Nine Inch Gremlins) >>> asciidoc-header Robert Dale On Tue, Sep 26, 2017 at 1:19 PM, Daniel Kuppitzwrote: > Ah, good find! That's the same issue I've had in the reference docs and > recipes. Fixed. > > Cheers, > Daniel > > > On Tue, Sep 26, 2017 at 10:08 AM, Robert Dale wrote: > > > git grep "== ==" > > > > Robert Dale > > > > On Tue, Sep 26, 2017 at 1:07 PM, Robert Dale wrote: > > > > > I see things like '== ==>6' in docs/src/upgrade/release-3.2. > > > x-incubating.html > > > > > > Robert Dale > > > > > > On Tue, Sep 26, 2017 at 12:57 PM, Robert Dale > wrote: > > > > > >> ^d > > >> > > >> Robert Dale > > >> > > >> On Tue, Sep 26, 2017 at 12:56 PM, Daniel Kuppitz > > wrote: > > >> > > >>> This is one. I pushed the changes to tp32/ and master/. I only found > > (and > > >>> fixed) 2 minor issues, where my script produced a little formatting > > mess. > > >>> > > >>> Cheers, > > >>> Daniel > > >>> > > >>> > > >>> On Tue, Sep 19, 2017 at 9:23 AM, Stephen Mallette < > > spmalle...@gmail.com> > > >>> wrote: > > >>> > > >>> > For whatever reason we started writing our headers in our docs > with: > > >>> > > > >>> > Header > > >>> > == > > >>> > > > >>> > Subheader > > >>> > - > > >>> > > > >>> > SubSubHeader > > >>> > > > >>> > > > >>> > as opposed to the alternative: > > >>> > > > >>> > = This Is A Header > > >>> > > > >>> > == Subheader > > >>> > > > >>> > === SubSubHeader > > >>> > > > >>> > The downside with the first approach is that it forces us to do two > > >>> things: > > >>> > > > >>> > 1. Remember the syntax for each header layer (which we all probably > > >>> know > > >>> > pretty well at this point, but still...) > > >>> > 2. Ensure that the width of the header formatting line is equal to > > the > > >>> > length of the text above it. > > >>> > > > >>> > That second point is a killer sometimes. While I've trained myself > to > > >>> be > > >>> > good at noticing that problem I know it's bitten us in releases > over > > >>> and > > >>> > over and over again. > > >>> > > > >>> > Anyway, I think going to the alternative formatting would be better > > and > > >>> > propose to do a full reformatting. Please let me know if there are > > any > > >>> > concerns about doing that - if I don't hear back in the next few > days > > >>> I'll > > >>> > proceed with the changes. > > >>> > > > >>> > Thanks, > > >>> > > > >>> > Stephen > > >>> > > > >>> > > >> > > >> > > > > > >
Re: [DISCUSS] Asciidoc Header Reformat
Ah, good find! That's the same issue I've had in the reference docs and recipes. Fixed. Cheers, Daniel On Tue, Sep 26, 2017 at 10:08 AM, Robert Dalewrote: > git grep "== ==" > > Robert Dale > > On Tue, Sep 26, 2017 at 1:07 PM, Robert Dale wrote: > > > I see things like '== ==>6' in docs/src/upgrade/release-3.2. > > x-incubating.html > > > > Robert Dale > > > > On Tue, Sep 26, 2017 at 12:57 PM, Robert Dale wrote: > > > >> ^d > >> > >> Robert Dale > >> > >> On Tue, Sep 26, 2017 at 12:56 PM, Daniel Kuppitz > wrote: > >> > >>> This is one. I pushed the changes to tp32/ and master/. I only found > (and > >>> fixed) 2 minor issues, where my script produced a little formatting > mess. > >>> > >>> Cheers, > >>> Daniel > >>> > >>> > >>> On Tue, Sep 19, 2017 at 9:23 AM, Stephen Mallette < > spmalle...@gmail.com> > >>> wrote: > >>> > >>> > For whatever reason we started writing our headers in our docs with: > >>> > > >>> > Header > >>> > == > >>> > > >>> > Subheader > >>> > - > >>> > > >>> > SubSubHeader > >>> > > >>> > > >>> > as opposed to the alternative: > >>> > > >>> > = This Is A Header > >>> > > >>> > == Subheader > >>> > > >>> > === SubSubHeader > >>> > > >>> > The downside with the first approach is that it forces us to do two > >>> things: > >>> > > >>> > 1. Remember the syntax for each header layer (which we all probably > >>> know > >>> > pretty well at this point, but still...) > >>> > 2. Ensure that the width of the header formatting line is equal to > the > >>> > length of the text above it. > >>> > > >>> > That second point is a killer sometimes. While I've trained myself to > >>> be > >>> > good at noticing that problem I know it's bitten us in releases over > >>> and > >>> > over and over again. > >>> > > >>> > Anyway, I think going to the alternative formatting would be better > and > >>> > propose to do a full reformatting. Please let me know if there are > any > >>> > concerns about doing that - if I don't hear back in the next few days > >>> I'll > >>> > proceed with the changes. > >>> > > >>> > Thanks, > >>> > > >>> > Stephen > >>> > > >>> > >> > >> > > >
Re: [DISCUSS] Asciidoc Header Reformat
git grep "== ==" Robert Dale On Tue, Sep 26, 2017 at 1:07 PM, Robert Dalewrote: > I see things like '== ==>6' in docs/src/upgrade/release-3.2. > x-incubating.html > > Robert Dale > > On Tue, Sep 26, 2017 at 12:57 PM, Robert Dale wrote: > >> ^d >> >> Robert Dale >> >> On Tue, Sep 26, 2017 at 12:56 PM, Daniel Kuppitz wrote: >> >>> This is one. I pushed the changes to tp32/ and master/. I only found (and >>> fixed) 2 minor issues, where my script produced a little formatting mess. >>> >>> Cheers, >>> Daniel >>> >>> >>> On Tue, Sep 19, 2017 at 9:23 AM, Stephen Mallette >>> wrote: >>> >>> > For whatever reason we started writing our headers in our docs with: >>> > >>> > Header >>> > == >>> > >>> > Subheader >>> > - >>> > >>> > SubSubHeader >>> > >>> > >>> > as opposed to the alternative: >>> > >>> > = This Is A Header >>> > >>> > == Subheader >>> > >>> > === SubSubHeader >>> > >>> > The downside with the first approach is that it forces us to do two >>> things: >>> > >>> > 1. Remember the syntax for each header layer (which we all probably >>> know >>> > pretty well at this point, but still...) >>> > 2. Ensure that the width of the header formatting line is equal to the >>> > length of the text above it. >>> > >>> > That second point is a killer sometimes. While I've trained myself to >>> be >>> > good at noticing that problem I know it's bitten us in releases over >>> and >>> > over and over again. >>> > >>> > Anyway, I think going to the alternative formatting would be better and >>> > propose to do a full reformatting. Please let me know if there are any >>> > concerns about doing that - if I don't hear back in the next few days >>> I'll >>> > proceed with the changes. >>> > >>> > Thanks, >>> > >>> > Stephen >>> > >>> >> >> >
Re: [DISCUSS] Asciidoc Header Reformat
I see things like '== ==>6' in docs/src/upgrade/release-3.2.x-incubating.html Robert Dale On Tue, Sep 26, 2017 at 12:57 PM, Robert Dalewrote: > ^d > > Robert Dale > > On Tue, Sep 26, 2017 at 12:56 PM, Daniel Kuppitz wrote: > >> This is one. I pushed the changes to tp32/ and master/. I only found (and >> fixed) 2 minor issues, where my script produced a little formatting mess. >> >> Cheers, >> Daniel >> >> >> On Tue, Sep 19, 2017 at 9:23 AM, Stephen Mallette >> wrote: >> >> > For whatever reason we started writing our headers in our docs with: >> > >> > Header >> > == >> > >> > Subheader >> > - >> > >> > SubSubHeader >> > >> > >> > as opposed to the alternative: >> > >> > = This Is A Header >> > >> > == Subheader >> > >> > === SubSubHeader >> > >> > The downside with the first approach is that it forces us to do two >> things: >> > >> > 1. Remember the syntax for each header layer (which we all probably know >> > pretty well at this point, but still...) >> > 2. Ensure that the width of the header formatting line is equal to the >> > length of the text above it. >> > >> > That second point is a killer sometimes. While I've trained myself to be >> > good at noticing that problem I know it's bitten us in releases over and >> > over and over again. >> > >> > Anyway, I think going to the alternative formatting would be better and >> > propose to do a full reformatting. Please let me know if there are any >> > concerns about doing that - if I don't hear back in the next few days >> I'll >> > proceed with the changes. >> > >> > Thanks, >> > >> > Stephen >> > >> > >
Re: [DISCUSS] Asciidoc Header Reformat
^d Robert Dale On Tue, Sep 26, 2017 at 12:56 PM, Daniel Kuppitzwrote: > This is one. I pushed the changes to tp32/ and master/. I only found (and > fixed) 2 minor issues, where my script produced a little formatting mess. > > Cheers, > Daniel > > > On Tue, Sep 19, 2017 at 9:23 AM, Stephen Mallette > wrote: > > > For whatever reason we started writing our headers in our docs with: > > > > Header > > == > > > > Subheader > > - > > > > SubSubHeader > > > > > > as opposed to the alternative: > > > > = This Is A Header > > > > == Subheader > > > > === SubSubHeader > > > > The downside with the first approach is that it forces us to do two > things: > > > > 1. Remember the syntax for each header layer (which we all probably know > > pretty well at this point, but still...) > > 2. Ensure that the width of the header formatting line is equal to the > > length of the text above it. > > > > That second point is a killer sometimes. While I've trained myself to be > > good at noticing that problem I know it's bitten us in releases over and > > over and over again. > > > > Anyway, I think going to the alternative formatting would be better and > > propose to do a full reformatting. Please let me know if there are any > > concerns about doing that - if I don't hear back in the next few days > I'll > > proceed with the changes. > > > > Thanks, > > > > Stephen > > >
Re: [DISCUSS] Asciidoc Header Reformat
This is one. I pushed the changes to tp32/ and master/. I only found (and fixed) 2 minor issues, where my script produced a little formatting mess. Cheers, Daniel On Tue, Sep 19, 2017 at 9:23 AM, Stephen Mallettewrote: > For whatever reason we started writing our headers in our docs with: > > Header > == > > Subheader > - > > SubSubHeader > > > as opposed to the alternative: > > = This Is A Header > > == Subheader > > === SubSubHeader > > The downside with the first approach is that it forces us to do two things: > > 1. Remember the syntax for each header layer (which we all probably know > pretty well at this point, but still...) > 2. Ensure that the width of the header formatting line is equal to the > length of the text above it. > > That second point is a killer sometimes. While I've trained myself to be > good at noticing that problem I know it's bitten us in releases over and > over and over again. > > Anyway, I think going to the alternative formatting would be better and > propose to do a full reformatting. Please let me know if there are any > concerns about doing that - if I don't hear back in the next few days I'll > proceed with the changes. > > Thanks, > > Stephen >
Re: [DISCUSS] Asciidoc Header Reformat
I ran into this problem previously as well; it's also interesting that the build will break if the length of the heading is much longer/shorter than the length of the underline (I think a difference of more than 2 characters causes a build error). Here is a Python and shell script I wrote to do the fix for an entire repo: #!/usr/bin/python import re import sys def main(argv): # http://asciidoctor.org/docs/asciidoc-recommended-practices/#section-titles patterns = [ (re.compile('^=+$'), '='), (re.compile('^-+$'), '=='), (re.compile('^~+$'), '==='), (re.compile('^\^+$'), ''), (re.compile('^\++$'), '='), ] with open(argv[1], 'r') as input_file: prev_line = None curr_line = None for line in input_file.readlines(): prev_line = curr_line curr_line = line if prev_line is None: continue for pattern, heading in patterns: if pattern.match(curr_line) and len(prev_line) == len(curr_line): sys.stdout.write('%s %s' % (heading, prev_line)) prev_line = None curr_line = None break if prev_line is not None: sys.stdout.write(prev_line) # end for if curr_line is not None: sys.stdout.write(curr_line) # end with if __name__ == '__main__': main(sys.argv) and here's a shell script driver: #!/bin/bash tmp="$(mktemp /tmp/asciidoc-cleanup.XX)" for file in *.txt ; do ./asciidoc.py $file > $tmp if [[ $? == 0 ]]; then mv $tmp $file else echo "error in $file" >&2 fi done YMMV, etc. Hope this helps! On Tue, Sep 19, 2017 at 12:23 PM, Stephen Mallettewrote: > For whatever reason we started writing our headers in our docs with: > > Header > == > > Subheader > - > > SubSubHeader > > > as opposed to the alternative: > > = This Is A Header > > == Subheader > > === SubSubHeader > > The downside with the first approach is that it forces us to do two things: > > 1. Remember the syntax for each header layer (which we all probably know > pretty well at this point, but still...) > 2. Ensure that the width of the header formatting line is equal to the > length of the text above it. > > That second point is a killer sometimes. While I've trained myself to be > good at noticing that problem I know it's bitten us in releases over and > over and over again. > > Anyway, I think going to the alternative formatting would be better and > propose to do a full reformatting. Please let me know if there are any > concerns about doing that - if I don't hear back in the next few days I'll > proceed with the changes. > > Thanks, > > Stephen >
[DISCUSS] Asciidoc Header Reformat
For whatever reason we started writing our headers in our docs with: Header == Subheader - SubSubHeader as opposed to the alternative: = This Is A Header == Subheader === SubSubHeader The downside with the first approach is that it forces us to do two things: 1. Remember the syntax for each header layer (which we all probably know pretty well at this point, but still...) 2. Ensure that the width of the header formatting line is equal to the length of the text above it. That second point is a killer sometimes. While I've trained myself to be good at noticing that problem I know it's bitten us in releases over and over and over again. Anyway, I think going to the alternative formatting would be better and propose to do a full reformatting. Please let me know if there are any concerns about doing that - if I don't hear back in the next few days I'll proceed with the changes. Thanks, Stephen