nex icon indicating copy to clipboard operation
nex copied to clipboard

update flex simple example link

Open denisacostaq opened this issue 6 years ago • 8 comments

flex project has been moved to github: http://flex.sourceforge.net/

denisacostaq avatar Jan 04 '19 21:01 denisacostaq

@denisacostaq If you're going to change the link, ok, but you should reflow to 80 chars.

purpleidea avatar Jan 04 '19 22:01 purpleidea

@denisacostaq If you're going to change the link, ok, but you should reflow to 80 chars.

thankyiu @purpleidea solved in 6fcf2d65665b7ee5cf28d542720b8b9978ee1517

denisacostaq avatar Jan 05 '19 23:01 denisacostaq

@denisacostaq Sorry I should have mentioned it earlier, but the convention is to not break links across lines, so you'll need to fix that, and you can rebase all your commits into a single one and force push.

purpleidea avatar Jan 05 '19 23:01 purpleidea

Still not correct, the link is broken, and you want only ONE commit at the end please.

purpleidea avatar Jan 06 '19 00:01 purpleidea

@purpleidea is right now?

denisacostaq avatar Jan 06 '19 00:01 denisacostaq

Almost:

The link should be:

[Foo does not break](URL)

IIRC

Foo

purpleidea avatar Jan 06 '19 00:01 purpleidea

bbl

purpleidea avatar Jan 06 '19 00:01 purpleidea

Almost:

The link should be:

[Foo does not break](URL)

IIRC

Foo

@purpleidea the example you shared me [text](url) is how GitHub Flavored Markdown Spec recommend, but if you look in the org REAME you can find this for example:

== Toy Pascal ==

The Flex manual also exhibits a http://flex.sourceforge.net/manual/Simple-Examples.html[scanner for a toy Pascal-like language],
though last I checked, its comment regex was a little buggy. Here is a

and I want to be consitent with this, I think the [] can be breake in multplie lines because it is not the link at all, but if you prefer this way you can fix this, including to fix the new link I am pointing in the pr and close this

denisacostaq avatar Jan 06 '19 00:01 denisacostaq