Re: [DISCUSS] Asciidoc Header Reformat

2017-09-26 Thread Robert Dale
Looks good!

Robert Dale

On Tue, Sep 26, 2017 at 1:38 PM, Daniel Kuppitz  wrote:

> 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

2017-09-26 Thread Daniel Kuppitz
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

2017-09-26 Thread Robert Dale
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 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

2017-09-26 Thread Robert Dale
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

2017-09-26 Thread Robert Dale
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

2017-09-26 Thread Daniel Kuppitz
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

2017-09-26 Thread Robert Dale
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 
>>> 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

2017-09-26 Thread Robert Dale
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

2017-09-26 Thread Robert Dale
^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

2017-09-26 Thread Daniel Kuppitz
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

2017-09-19 Thread Misha Brukman
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 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
>