Re: PR #363

[Mario Brandt]

+1 for the change. It is still pretty readable on any edtor.


-- Originalnachricht --
Von rbo...@rcbowen.com
An "dev@httpd.apache.org" 
Datum 24.01.2024 16:06:43
Betreff PR #363


I've been poking at some of our open PRs looking for docs-only changes
that we can possibly clean up.

PR #363 - https://github.com/apache/httpd/pull/363 - converts README
from plaintext to markdown, and makes no other changes. This makes it
prettier on GitHub.

I almost just CTR'ed it, but it occurred to me that someone might find
this objectionable, so I thought I'd ask first.

Re: PR #363

[Yann Ylavic]

+1

On Thu, Jan 25, 2024 at 3:45 PM Eric Covener  wrote:
>
> I wouldn't mind move/rename to README.md
>
> On Thu, Jan 25, 2024, 10:40 AM Joe Orton  wrote:
>>
>> On Thu, Jan 25, 2024 at 08:12:24AM +0100, Ruediger Pluem wrote:
>> > Tried it in r1915391 and it seems to work. Not sure if there are
>> > general downsides / objections with regards to symlinks in our
>> > repository. But trunk is CTR :-).
>>
>> Oh, that looks really nice. +1
>>
>> Thanks to you, Rich, and Mayank Patil.
>>
>> Regards, Joe
>>

Re: PR #363

[Eric Covener]

I wouldn't mind move/rename to README.md

On Thu, Jan 25, 2024, 10:40 AM Joe Orton  wrote:

> On Thu, Jan 25, 2024 at 08:12:24AM +0100, Ruediger Pluem wrote:
> > Tried it in r1915391 and it seems to work. Not sure if there are
> > general downsides / objections with regards to symlinks in our
> > repository. But trunk is CTR :-).
>
> Oh, that looks really nice. +1
>
> Thanks to you, Rich, and Mayank Patil.
>
> Regards, Joe
>
>

Re: PR #363

[Joe Orton]

On Thu, Jan 25, 2024 at 08:12:24AM +0100, Ruediger Pluem wrote:
> Tried it in r1915391 and it seems to work. Not sure if there are 
> general downsides / objections with regards to symlinks in our 
> repository. But trunk is CTR :-).

Oh, that looks really nice. +1

Thanks to you, Rich, and Mayank Patil.

Regards, Joe

Re: PR #363

[Ruediger Pluem]

On 1/25/24 8:04 AM, Ruediger Pluem wrote:
> 
> 
> On 1/24/24 11:31 PM, Rich Bowen wrote:
>> Well, it turns out that in order for this to actually work, we would need to 
>> rename README to README.md, so I'm inclined to revert
>> this change unless someone thinks differently.
> 
> How about using a symlink? Keep README and create a README.md symlink to it?
> But I am not sure if github handles this correctly.

Tried it in r1915391 and it seems to work. Not sure if there are general 
downsides / objections with regards to symlinks in our
repository. But trunk is CTR :-).

Regards

Rüdiger

Re: PR #363

[Ruediger Pluem]

On 1/24/24 11:31 PM, Rich Bowen wrote:
> Well, it turns out that in order for this to actually work, we would need to 
> rename README to README.md, so I'm inclined to revert
> this change unless someone thinks differently.

How about using a symlink? Keep README and create a README.md symlink to it?
But I am not sure if github handles this correctly.

Regards

Rüdiger

Re: PR #363

[Rich Bowen]

Well, it turns out that in order for this to actually work, we would need
to rename README to README.md, so I'm inclined to revert this change unless
someone thinks differently.

Rich

On Wed, Jan 24, 2024, 10:06  wrote:

> I've been poking at some of our open PRs looking for docs-only changes
> that we can possibly clean up.
>
> PR #363 - https://github.com/apache/httpd/pull/363 - converts README
> from plaintext to markdown, and makes no other changes. This makes it
> prettier on GitHub.
>
> I almost just CTR'ed it, but it occurred to me that someone might find
> this objectionable, so I thought I'd ask first.
>

Re: PR #363

[Yann Ylavic]

On Wed, Jan 24, 2024 at 5:07 PM  wrote:
>
> On Wed, 2024-01-24 at 17:01 +0100, Yann Ylavic wrote:
> > On Wed, Jan 24, 2024 at 4:56 PM Mads Toftum  wrote:
> > >
> > > On Wed, Jan 24, 2024 at 10:06:43AM -0500, rbo...@rcbowen.com wrote:
> > > > I've been poking at some of our open PRs looking for docs-only
> > > > changes
> > > > that we can possibly clean up.
> > > >
> > > > PR #363 - https://github.com/apache/httpd/pull/363 - converts
> > > > README
> > > > from plaintext to markdown, and makes no other changes. This
> > > > makes it
> > > > prettier on GitHub.
> > > >
> > > > I almost just CTR'ed it, but it occurred to me that someone might
> > > > find
> > > > this objectionable, so I thought I'd ask first.
> > >
> > > I don't have strong opinions on this, but we'll end up with a
> > > READNE
> > > that's less readable for those not viewing it through github.
> >
> > Maybe we could keep the line breaks as before for the text to keep it
> > as readable as before w/o a markdown viewer?
>
> Yeah, I've done that in my local copy of the patch.
>
> So this is basically changing:
>
> Whossname
> -
>
> to
>
> ## Whossname
>
> Which really doesn't change much in plain text mode.

Agreed, the main readability issue (in plain text mode) is long lines
IMHO, headings/links do not read so bad.

Re: PR #363

[rbowen]

On Wed, 2024-01-24 at 17:01 +0100, Yann Ylavic wrote:
> On Wed, Jan 24, 2024 at 4:56 PM Mads Toftum  wrote:
> > 
> > On Wed, Jan 24, 2024 at 10:06:43AM -0500, rbo...@rcbowen.com wrote:
> > > I've been poking at some of our open PRs looking for docs-only
> > > changes
> > > that we can possibly clean up.
> > > 
> > > PR #363 - https://github.com/apache/httpd/pull/363 - converts
> > > README
> > > from plaintext to markdown, and makes no other changes. This
> > > makes it
> > > prettier on GitHub.
> > > 
> > > I almost just CTR'ed it, but it occurred to me that someone might
> > > find
> > > this objectionable, so I thought I'd ask first.
> > 
> > I don't have strong opinions on this, but we'll end up with a
> > READNE
> > that's less readable for those not viewing it through github.
> 
> Maybe we could keep the line breaks as before for the text to keep it
> as readable as before w/o a markdown viewer?

Yeah, I've done that in my local copy of the patch.

So this is basically changing:

Whossname
-

to

## Whossname

Which really doesn't change much in plain text mode.

Re: PR #363

[Joe Schaefer]

With GFM, each line break separates paragraphs.

Joe Schaefer, Ph.D.

Orion - The Enterprise Jamstack Wiki 

954.253.3732 




On Wed, Jan 24, 2024 at 11:01 AM Yann Ylavic  wrote:

> On Wed, Jan 24, 2024 at 4:56 PM Mads Toftum  wrote:
> >
> > On Wed, Jan 24, 2024 at 10:06:43AM -0500, rbo...@rcbowen.com wrote:
> > > I've been poking at some of our open PRs looking for docs-only changes
> > > that we can possibly clean up.
> > >
> > > PR #363 - https://github.com/apache/httpd/pull/363 - converts README
> > > from plaintext to markdown, and makes no other changes. This makes it
> > > prettier on GitHub.
> > >
> > > I almost just CTR'ed it, but it occurred to me that someone might find
> > > this objectionable, so I thought I'd ask first.
> >
> > I don't have strong opinions on this, but we'll end up with a READNE
> > that's less readable for those not viewing it through github.
>
> Maybe we could keep the line breaks as before for the text to keep it
> as readable as before w/o a markdown viewer?
>
>
> Regards;
> Yann.
>

Re: PR #363

[Yann Ylavic]

On Wed, Jan 24, 2024 at 4:56 PM Mads Toftum  wrote:
>
> On Wed, Jan 24, 2024 at 10:06:43AM -0500, rbo...@rcbowen.com wrote:
> > I've been poking at some of our open PRs looking for docs-only changes
> > that we can possibly clean up.
> >
> > PR #363 - https://github.com/apache/httpd/pull/363 - converts README
> > from plaintext to markdown, and makes no other changes. This makes it
> > prettier on GitHub.
> >
> > I almost just CTR'ed it, but it occurred to me that someone might find
> > this objectionable, so I thought I'd ask first.
>
> I don't have strong opinions on this, but we'll end up with a READNE
> that's less readable for those not viewing it through github.

Maybe we could keep the line breaks as before for the text to keep it
as readable as before w/o a markdown viewer?


Regards;
Yann.

Re: PR #363

[Mads Toftum]

On Wed, Jan 24, 2024 at 10:06:43AM -0500, rbo...@rcbowen.com wrote:
> I've been poking at some of our open PRs looking for docs-only changes
> that we can possibly clean up.
> 
> PR #363 - https://github.com/apache/httpd/pull/363 - converts README
> from plaintext to markdown, and makes no other changes. This makes it
> prettier on GitHub.
> 
> I almost just CTR'ed it, but it occurred to me that someone might find
> this objectionable, so I thought I'd ask first.

I don't have strong opinions on this, but we'll end up with a READNE
that's less readable for those not viewing it through github.

vh

Mads Toftum
-- 
http://flickr.com/photos/q42/