[GitHub] asfgit closed pull request #2: Replace module-markdown pegdown parser to flexmark-java

asfgit closed pull request #2: Replace module-markdown pegdown parser to flexmark-java URL: https://github.com/apache/maven-doxia/pull/2 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this

[GitHub] bhav0904 commented on issue #2: [JXR-100] Fix cross reference generation where there are spaces befor..

bhav0904 commented on issue #2: [JXR-100] Fix cross reference generation where there are spaces befor.. URL: https://github.com/apache/maven-jxr/pull/2#issuecomment-347204846 It?s not, it was an unused method that I found and wanted to clean up. On Mon, 27 Nov 2017 at 9:31 AM,

[GitHub] rfscholte commented on a change in pull request #2: [JXR-100] Fix cross reference generation where there are spaces befor..

rfscholte commented on a change in pull request #2: [JXR-100] Fix cross reference generation where there are spaces befor.. URL: https://github.com/apache/maven-jxr/pull/2#discussion_r153211196 ## File path: maven-jxr/src/main/java/org/apache/maven/jxr/JavaCodeTransform.java

[GitHub] michael-o commented on a change in pull request #5: [DOXIA-534] Migrate logging to Sl4j

michael-o commented on a change in pull request #5: [DOXIA-534] Migrate logging to Sl4j URL: https://github.com/apache/maven-doxia/pull/5#discussion_r153050140 ## File path: pom.xml ## @@ -185,15 +180,33 @@ under the License. org.codehaus.plexus

[GitHub] michael-o commented on issue #5: [DOXIA-534] Migrate logging to Sl4j

michael-o commented on issue #5: [DOXIA-534] Migrate logging to Sl4j URL: https://github.com/apache/maven-doxia/pull/5#issuecomment-346960574 @slachiewicz Can you spawn another ticket and create a new PR? We will leave this one open.

[GitHub] michael-o commented on issue #5: [DOXIA-534] Migrate logging to Sl4j

michael-o commented on issue #5: [DOXIA-534] Migrate logging to Sl4j URL: https://github.com/apache/maven-doxia/pull/5#issuecomment-346953287 @hboutemy Looks good to me. Tests pending... This is an automated message from the

[GitHub] slachiewicz opened a new pull request #5: [DOXIA-534] Migrate logging to Sl4j

slachiewicz opened a new pull request #5: [DOXIA-534] Migrate logging to Sl4j URL: https://github.com/apache/maven-doxia/pull/5 Please check. Current build fails with cirr error - due to interfaces change. This is an

[GitHub] michael-o commented on a change in pull request #5: [DOXIA-534] Migrate logging to Sl4j

michael-o commented on a change in pull request #5: [DOXIA-534] Migrate logging to Sl4j URL: https://github.com/apache/maven-doxia/pull/5#discussion_r153050104 ## File path: doxia-modules/doxia-module-fo/pom.xml ## @@ -78,7 +72,7 @@ under the License.

[GitHub] hboutemy commented on issue #5: [DOXIA-534] Migrate logging to Sl4j

hboutemy commented on issue #5: [DOXIA-534] Migrate logging to Sl4j URL: https://github.com/apache/maven-doxia/pull/5#issuecomment-346957484 +1 This is an automated message from the Apache Git Service. To respond to the

[GitHub] michael-o commented on issue #5: [DOXIA-534] Migrate logging to Sl4j

michael-o commented on issue #5: [DOXIA-534] Migrate logging to Sl4j URL: https://github.com/apache/maven-doxia/pull/5#issuecomment-346955331 Why not deprecate in 1.8 and remove with 2.0? This is an automated message from the

[GitHub] hboutemy commented on issue #5: [DOXIA-534] Migrate logging to Sl4j

hboutemy commented on issue #5: [DOXIA-534] Migrate logging to Sl4j URL: https://github.com/apache/maven-doxia/pull/5#issuecomment-346954321 I'm not in favor of removing doxia-logging: deprecating and modifying it to use slf4j would be more compatible I know there are not so much Doxia

[GitHub] michael-o commented on a change in pull request #5: [DOXIA-534] Migrate logging to Sl4j

michael-o commented on a change in pull request #5: [DOXIA-534] Migrate logging to Sl4j URL: https://github.com/apache/maven-doxia/pull/5#discussion_r153050158 ## File path: doxia-modules/doxia-module-fo/src/main/java/org/apache/maven/doxia/module/fo/FoAggregateSink.java

[GitHub] slachiewicz commented on a change in pull request #5: [DOXIA-534] Migrate logging to Sl4j

slachiewicz commented on a change in pull request #5: [DOXIA-534] Migrate logging to Sl4j URL: https://github.com/apache/maven-doxia/pull/5#discussion_r153053842 ## File path: doxia-modules/doxia-module-fo/src/main/java/org/apache/maven/doxia/module/fo/FoAggregateSink.java

[GitHub] slachiewicz commented on a change in pull request #5: [DOXIA-534] Migrate logging to Sl4j

slachiewicz commented on a change in pull request #5: [DOXIA-534] Migrate logging to Sl4j URL: https://github.com/apache/maven-doxia/pull/5#discussion_r153053853 ## File path: pom.xml ## @@ -185,15 +180,33 @@ under the License. org.codehaus.plexus

[GitHub] slachiewicz opened a new pull request #6: [DOXIA-535] Upgrade dependences and plugin versions

slachiewicz opened a new pull request #6: [DOXIA-535] Upgrade dependences and plugin versions URL: https://github.com/apache/maven-doxia/pull/6 This is an automated message from the Apache Git Service. To respond to the

[GitHub] hboutemy commented on issue #6: [DOXIA-566] Upgrade dependences and plugin versions

hboutemy commented on issue #6: [DOXIA-566] Upgrade dependences and plugin versions URL: https://github.com/apache/maven-doxia/pull/6#issuecomment-347043431 please don't update plugins: this will happen with parent pom. AFAIK, no hurry. upgrade of dependencies is what I'd like to

[GitHub] bhav0904 opened a new pull request #2: [JXR-100] Fix cross reference generation where there are spaces befor..

bhav0904 opened a new pull request #2: [JXR-100] Fix cross reference generation where there are spaces befor.. URL: https://github.com/apache/maven-jxr/pull/2 ?e or after parantheses. This is an automated message from the

[GitHub] bhav0904 opened a new pull request #1: Fix typos in AbstractJxrReport.

bhav0904 opened a new pull request #1: Fix typos in AbstractJxrReport. URL: https://github.com/apache/maven-jxr/pull/1 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] hboutemy commented on issue #1: Fix typos in AbstractJxrReport.

hboutemy commented on issue #1: Fix typos in AbstractJxrReport. URL: https://github.com/apache/maven-jxr/pull/1#issuecomment-343755961 merged in https://github.com/apache/maven-jxr/commit/b1bc166e6335f1526ef902f49ab59b32f3475ab1 thank you

[GitHub] hboutemy closed pull request #1: Fix typos in AbstractJxrReport.

hboutemy closed pull request #1: Fix typos in AbstractJxrReport. URL: https://github.com/apache/maven-jxr/pull/1 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request

[GitHub] hboutemy commented on issue #27: Fixed wrong page title for Dependency Convergence

hboutemy commented on issue #27: Fixed wrong page title for Dependency Convergence URL: https://github.com/apache/maven-enforcer/pull/27#issuecomment-341901865 merged in https://github.com/apache/maven-enforcer/commit/a7b656d2b60fb9cf316090cb3474e6477cff1849 thank you

[GitHub] hboutemy closed pull request #27: Fixed wrong page title for Dependency Convergence

hboutemy closed pull request #27: Fixed wrong page title for Dependency Convergence URL: https://github.com/apache/maven-enforcer/pull/27 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this

[GitHub] arend-von-reinersdorff opened a new pull request #27: Fixed wrong page title for Dependency Convergence

arend-von-reinersdorff opened a new pull request #27: Fixed wrong page title for Dependency Convergence URL: https://github.com/apache/maven-enforcer/pull/27 Check page title here: http://maven.apache.org/enforcer/enforcer-rules/dependencyConvergence.html

[GitHub] hboutemy commented on issue #27: Fixed wrong page title for Dependency Convergence

hboutemy commented on issue #27: Fixed wrong page title for Dependency Convergence URL: https://github.com/apache/maven-enforcer/pull/27#issuecomment-341901250 sorry, this was automatically closed when migrating to Git and deleting old trunk branch reopening

[GitHub] oleg-nenashev opened a new pull request #25: MENFORCER-276 - Support ignoring dependency scopes in RequireUpperBoundDeps

oleg-nenashev opened a new pull request #25: MENFORCER-276 - Support ignoring dependency scopes in RequireUpperBoundDeps URL: https://github.com/apache/maven-enforcer/pull/25 We have recently adopted `RequireUpperBoundDeps` in the Jenkins project

[GitHub] jtnord opened a new pull request #28: [MENFORCER-281] added IT to show the issue.

jtnord opened a new pull request #28: [MENFORCER-281] added IT to show the issue. URL: https://github.com/apache/maven-enforcer/pull/28 Added an IT to demonstrate [MENFORCER-281](https://issues.apache.org/jira/browse/MENFORCER-281) Fails with the following which is slightly

[GitHub] oleg-nenashev opened a new pull request #26: Add Unit tests for RequireUpperBoundDeps

oleg-nenashev opened a new pull request #26: Add Unit tests for RequireUpperBoundDeps URL: https://github.com/apache/maven-enforcer/pull/26 I decoupled the test improvements commit from #25. It just adds new tests for the existing functionality. @reviewbybees @jglick @stephenc

[GitHub] slachiewicz opened a new pull request #29: Update to Mockito 2.11

slachiewicz opened a new pull request #29: Update to Mockito 2.11 URL: https://github.com/apache/maven-enforcer/pull/29 Fixes MENFORCER-278 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] jgangemi commented on issue #4: [MRELEASE-798] Commit additional files with release plugin

jgangemi commented on issue #4: [MRELEASE-798] Commit additional files with release plugin URL: https://github.com/apache/maven-release/pull/4#issuecomment-341118397 so this issue is being closed outright and not even considered?!! an explanation as to why would be real nice.

[GitHub] jgangemi commented on issue #4: [MRELEASE-798] Commit additional files with release plugin

jgangemi commented on issue #4: [MRELEASE-798] Commit additional files with release plugin URL: https://github.com/apache/maven-release/pull/4#issuecomment-341118397 so this issue is being closed outright and not even considered?!!

[GitHub] emenaceb opened a new pull request #4: [MRELEASE-798] Commit additional files with release plugin

emenaceb opened a new pull request #4: [MRELEASE-798] Commit additional files with release plugin URL: https://github.com/apache/maven-release/pull/4 Ability to commit additional files on prepare This is an automated

[GitHub] hboutemy commented on issue #4: [MRELEASE-798] Commit additional files with release plugin

hboutemy commented on issue #4: [MRELEASE-798] Commit additional files with release plugin URL: https://github.com/apache/maven-release/pull/4#issuecomment-341149983 uh! sorry, we just migrated and it seems GitHub integration did unexpected things (and tells I did it, since I deleted

[GitHub] arend-von-reinersdorff commented on issue #27: Fixed wrong page title for Dependency Convergence

arend-von-reinersdorff commented on issue #27: Fixed wrong page title for Dependency Convergence URL: https://github.com/apache/maven-enforcer/pull/27#issuecomment-341898122 What was the reason to close this? This is an

[GitHub] slachiewicz commented on issue #6: [DOXIA-566] Upgrade dependences and plugin versions

slachiewicz commented on issue #6: [DOXIA-566] Upgrade dependences and plugin versions URL: https://github.com/apache/maven-doxia/pull/6#issuecomment-348937905 ok, plugin updates removed This is an automated message from the

[GitHub] hboutemy commented on issue #6: [DOXIA-566] Upgrade dependences and plugin versions

hboutemy commented on issue #6: [DOXIA-566] Upgrade dependences and plugin versions URL: https://github.com/apache/maven-doxia/pull/6#issuecomment-349487617 PR merged thank you This is an automated message from the Apache

[GitHub] hboutemy closed pull request #6: [DOXIA-566] Upgrade dependences and plugin versions

hboutemy closed pull request #6: [DOXIA-566] Upgrade dependences and plugin versions URL: https://github.com/apache/maven-doxia/pull/6 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] don-vip opened a new pull request #9: upgrade to ASM 6.0 (final)

don-vip opened a new pull request #9: upgrade to ASM 6.0 (final) URL: https://github.com/apache/maven-plugin-tools/pull/9 The ALPHA dependency causes problems to projects using Enforcer rules, see https://github.com/jenkinsci/jacoco-plugin/pull/92#issuecomment-349457441

[GitHub] asfgit closed pull request #20: MENFORCER-195 - use DependencyGraphBuilder API instead of DependencyTreeBuilder

asfgit closed pull request #20: MENFORCER-195 - use DependencyGraphBuilder API instead of DependencyTreeBuilder URL: https://github.com/apache/maven-enforcer/pull/20 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] asfgit closed pull request #26: Add Unit tests for RequireUpperBoundDeps

asfgit closed pull request #26: Add Unit tests for RequireUpperBoundDeps URL: https://github.com/apache/maven-enforcer/pull/26 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign

[GitHub] asfgit closed pull request #17: Added support for reading banned exclusions from a URL

asfgit closed pull request #17: Added support for reading banned exclusions from a URL URL: https://github.com/apache/maven-enforcer/pull/17 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As

[GitHub] asfgit closed pull request #27: Fixed wrong page title for Dependency Convergence

asfgit closed pull request #27: Fixed wrong page title for Dependency Convergence URL: https://github.com/apache/maven-enforcer/pull/27 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is

[GitHub] asfgit closed pull request #21: Add rule: banProfiles

asfgit closed pull request #21: Add rule: banProfiles URL: https://github.com/apache/maven-enforcer/pull/21 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from

[GitHub] asfgit closed pull request #6: add a IT case for bannedDependencies in a multi module project

asfgit closed pull request #6: add a IT case for bannedDependencies in a multi module project URL: https://github.com/apache/maven-enforcer/pull/6 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

[GitHub] asfgit closed pull request #28: [MENFORCER-281] added IT to show the issue.

asfgit closed pull request #28: [MENFORCER-281] added IT to show the issue. URL: https://github.com/apache/maven-enforcer/pull/28 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] asfgit closed pull request #16: MENFORCER-241 allow ignoring minor and patch level mismatches

asfgit closed pull request #16: MENFORCER-241 allow ignoring minor and patch level mismatches URL: https://github.com/apache/maven-enforcer/pull/16 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

[GitHub] asfgit closed pull request #7: MPLUGIN-294 - 'report' mojo should use 'extractors' configuration parameter

asfgit closed pull request #7: MPLUGIN-294 - 'report' mojo should use 'extractors' configuration parameter URL: https://github.com/apache/maven-plugin-tools/pull/7 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

[GitHub] asfgit closed pull request #6: For Groovy/Scala Mojos: Allow Mojo documentation in annotations, too.

asfgit closed pull request #6: For Groovy/Scala Mojos: Allow Mojo documentation in annotations, too. URL: https://github.com/apache/maven-plugin-tools/pull/6 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] asfgit closed pull request #5: Preserving the annotations at runtime is very useful for testing

asfgit closed pull request #5: Preserving the annotations at runtime is very useful for testing URL: https://github.com/apache/maven-plugin-tools/pull/5 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] asfgit closed pull request #4: Update to ASM5 opscode.

asfgit closed pull request #4: Update to ASM5 opscode. URL: https://github.com/apache/maven-plugin-tools/pull/4 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request

[GitHub] theit opened a new pull request #19: Fix for MRELEASE-835: Automatically resolve snapshot dependencies in release:prepare

theit opened a new pull request #19: Fix for MRELEASE-835: Automatically resolve snapshot dependencies in release:prepare URL: https://github.com/apache/maven-release/pull/19 This pull requests adds support to release:prepare to let it automatically resolve snapshot dependencies without

[GitHub] hboutemy closed pull request #4: [MRELEASE-798] Commit additional files with release plugin

hboutemy closed pull request #4: [MRELEASE-798] Commit additional files with release plugin URL: https://github.com/apache/maven-release/pull/4 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As

[GitHub] hboutemy closed pull request #13: MRELEASE-980 Allow commit comments to be controlled

hboutemy closed pull request #13: MRELEASE-980 Allow commit comments to be controlled URL: https://github.com/apache/maven-release/pull/13 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this

[GitHub] minfrin opened a new pull request #10: XHTML5 support for maven-doxia

minfrin opened a new pull request #10: XHTML5 support for maven-doxia URL: https://github.com/apache/maven-doxia/pull/10 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] slachiewicz opened a new pull request #12: DOXIA-576 Upgrade Http Components to 4.4.5

slachiewicz opened a new pull request #12: DOXIA-576 Upgrade Http Components to 4.4.5 URL: https://github.com/apache/maven-doxia/pull/12 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] comtention commented on issue #11: Propagate exceptions to caller

comtention commented on issue #11: Propagate exceptions to caller URL: https://github.com/apache/maven-doxia/pull/11#issuecomment-398056960 I have exactly the same issue, it is really blocking for me. This is an automated

[GitHub] ghilainm opened a new pull request #11: Propagate exceptions to caller

ghilainm opened a new pull request #11: Propagate exceptions to caller URL: https://github.com/apache/maven-doxia/pull/11 The exceptions are swallowed this is really unpractical when using the library (example building a site using mvn-site plugin).

[GitHub] olamy commented on issue #19: Fix for MRELEASE-835: Automatically resolve snapshot dependencies in release:prepare

olamy commented on issue #19: Fix for MRELEASE-835: Automatically resolve snapshot dependencies in release:prepare URL: https://github.com/apache/maven-release/pull/19#issuecomment-339855892 @theit I'm happy with those changes. Except the lack of test. Is there anything you can do? A

[GitHub] olamy commented on issue #19: Fix for MRELEASE-835: Automatically resolve snapshot dependencies in release:prepare

olamy commented on issue #19: Fix for MRELEASE-835: Automatically resolve snapshot dependencies in release:prepare URL: https://github.com/apache/maven-release/pull/19#issuecomment-339855892 @theit I'm happy with this changes. Except the lack of test. Is there anything you can do? A

[GitHub] ctubbsii commented on issue #4: [MRELEASE-798] Commit additional files with release plugin

ctubbsii commented on issue #4: [MRELEASE-798] Commit additional files with release plugin URL: https://github.com/apache/maven-release/pull/4#issuecomment-339842030 +1 to this feature request I'm trying to update an OSGi manifest with the version derived from the version in the

[GitHub] asfbot commented on issue #6: add a IT case for bannedDependencies in a multi module project

asfbot commented on issue #6: add a IT case for bannedDependencies in a multi module project URL: https://github.com/apache/maven-enforcer/pull/6#issuecomment-339636554 Stephen Connolly on dev@maven.apache.org replies: could we get these sent to commits@m.a.o rather than dev@m.a.o

[GitHub] hazendaz commented on a change in pull request #5: [DOXIA-534] Migrate logging to Sl4j

hazendaz commented on a change in pull request #5: [DOXIA-534] Migrate logging to Sl4j URL: https://github.com/apache/maven-doxia/pull/5#discussion_r159147868 ## File path: doxia-modules/doxia-module-fo/src/main/java/org/apache/maven/doxia/module/fo/FoAggregateSink.java

[GitHub] hboutemy commented on issue #7: [DOXIA-409] Upgrade to FOP 2.2

hboutemy commented on issue #7: [DOXIA-409] Upgrade to FOP 2.2 URL: https://github.com/apache/maven-doxia/pull/7#issuecomment-357500503 interesting: did you try the result of this change with maven-pdf-plugin or other tools/plugins that produce pdf using FO?

[GitHub] slachiewicz commented on issue #7: [DOXIA-409] Upgrade to FOP 2.2

slachiewicz commented on issue #7: [DOXIA-409] Upgrade to FOP 2.2 URL: https://github.com/apache/maven-doxia/pull/7#issuecomment-363521508 Yes, i run tests inside doxia (some minor warnings) and today also tests with maven-pdf-plugin and all looks good.

[GitHub] slachiewicz opened a new pull request #9: [DOXIA-570] Escape links to xml based figureGraphics image elements

slachiewicz opened a new pull request #9: [DOXIA-570] Escape links to xml based figureGraphics image elements URL: https://github.com/apache/maven-doxia/pull/9 This is an automated message from the Apache Git Service. To

[GitHub] slachiewicz opened a new pull request #8: [DOXIA-534] Migrate logging to slf4j and deprecate doxia-logging-api

slachiewicz opened a new pull request #8: [DOXIA-534] Migrate logging to slf4j and deprecate doxia-logging-api URL: https://github.com/apache/maven-doxia/pull/8 New version that deprecates doxia-logging This is an automated

[GitHub] slachiewicz commented on issue #5: [DOXIA-534] Migrate logging to Sl4j

slachiewicz commented on issue #5: [DOXIA-534] Migrate logging to Sl4j URL: https://github.com/apache/maven-doxia/pull/5#issuecomment-364775373 New version with deprecation in #8 This is an automated message from the Apache

[GitHub] slachiewicz closed pull request #5: [DOXIA-534] Migrate logging to Sl4j

slachiewicz closed pull request #5: [DOXIA-534] Migrate logging to Sl4j URL: https://github.com/apache/maven-doxia/pull/5 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] hboutemy commented on issue #7: [DOXIA-409] Upgrade to FOP 2.2

hboutemy commented on issue #7: [DOXIA-409] Upgrade to FOP 2.2 URL: https://github.com/apache/maven-doxia/pull/7#issuecomment-363692292 ok, let's have a try I merged this PR and will update maven-pdf-plugin to use the new SNAPSHOT then please ask on the dev mailing list for people to

[GitHub] hboutemy closed pull request #7: [DOXIA-409] Upgrade to FOP 2.2

hboutemy closed pull request #7: [DOXIA-409] Upgrade to FOP 2.2 URL: https://github.com/apache/maven-doxia/pull/7 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request

[GitHub] michael-o commented on issue #11: Propagate exceptions to caller

michael-o commented on issue #11: Propagate exceptions to caller URL: https://github.com/apache/maven-doxia/pull/11#issuecomment-415997595 A JIRA issue please and I will merge. This is an automated message from the Apache Git

[GitHub] slachiewicz opened a new pull request #13: [DOXIA-497] Fix rendering links and paragraphs inside tables

slachiewicz opened a new pull request #13: [DOXIA-497] Fix rendering links and paragraphs inside tables URL: https://github.com/apache/maven-doxia/pull/13 This is an automated message from the Apache Git Service. To respond

[GitHub] asfgit closed pull request #13: [DOXIA-497] Fix rendering links and paragraphs inside tables

asfgit closed pull request #13: [DOXIA-497] Fix rendering links and paragraphs inside tables URL: https://github.com/apache/maven-doxia/pull/13 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As

[GitHub] slachiewicz opened a new pull request #7: [DOXIA-409] Upgrade to FOP 2.2

slachiewicz opened a new pull request #7: [DOXIA-409] Upgrade to FOP 2.2 URL: https://github.com/apache/maven-doxia/pull/7 Fix by: Michael Simacek

[GitHub] hboutemy commented on a change in pull request #11: Propagate macro related exceptions to caller in xhtml parser

hboutemy commented on a change in pull request #11: Propagate macro related exceptions to caller in xhtml parser URL: https://github.com/apache/maven-doxia/pull/11#discussion_r249715464 ## File path:

[GitHub] hboutemy commented on a change in pull request #11: Propagate exceptions to caller

hboutemy commented on a change in pull request #11: Propagate exceptions to caller URL: https://github.com/apache/maven-doxia/pull/11#discussion_r249714608 ## File path: doxia-modules/doxia-module-xhtml/src/main/java/org/apache/maven/doxia/module/xhtml/XhtmlParser.java ##

[GitHub] hboutemy commented on issue #11: Propagate macro related exceptions to caller in xhtml parser

hboutemy commented on issue #11: Propagate macro related exceptions to caller in xhtml parser URL: https://github.com/apache/maven-doxia/pull/11#issuecomment-456344828 FYI, Jira issue created: https://issues.apache.org/jira/browse/DOXIA-585

[GitHub] hboutemy commented on issue #11: Propagate macro related exceptions to caller in xhtml parser

hboutemy commented on issue #11: Propagate macro related exceptions to caller in xhtml parser URL: https://github.com/apache/maven-doxia/pull/11#issuecomment-456790819 fixed in d977f4e4f4a9f7d5549f25daa56d3cd0f756023a this not only keeps the exception in case of macro execution failure

[GitHub] hboutemy closed pull request #11: Propagate macro related exceptions to caller in xhtml parser

hboutemy closed pull request #11: Propagate macro related exceptions to caller in xhtml parser URL: https://github.com/apache/maven-doxia/pull/11 This is an automated message from the Apache Git Service. To respond to the

[GitHub] jmones opened a new pull request #14: [DOXIA-584] Rewrite .md and .markdown links to .html

jmones opened a new pull request #14: [DOXIA-584] Rewrite .md and .markdown links to .html URL: https://github.com/apache/maven-doxia/pull/14 This implementation supports rewriting the following types of links: (link to a)[a.md] (link to xxx section in a)[a.md#xxx] (link to

[GitHub] slachiewicz commented on issue #8: [DOXIA-534] Migrate logging to slf4j and deprecate doxia-logging-api

slachiewicz commented on issue #8: [DOXIA-534] Migrate logging to slf4j and deprecate doxia-logging-api URL: https://github.com/apache/maven-doxia/pull/8#issuecomment-457848628 @hboutemy ping? This is an automated message

[GitHub] ghilainm commented on a change in pull request #11: Propagate macro related exceptions to caller in xhtml parser

ghilainm commented on a change in pull request #11: Propagate macro related exceptions to caller in xhtml parser URL: https://github.com/apache/maven-doxia/pull/11#discussion_r249834594 ## File path:

[GitHub] ghilainm commented on a change in pull request #11: Propagate macro related exceptions to caller in xhtml parser

ghilainm commented on a change in pull request #11: Propagate macro related exceptions to caller in xhtml parser URL: https://github.com/apache/maven-doxia/pull/11#discussion_r249834862 ## File path:

[GitHub] hboutemy commented on a change in pull request #11: Propagate macro related exceptions to caller in xhtml parser

hboutemy commented on a change in pull request #11: Propagate macro related exceptions to caller in xhtml parser URL: https://github.com/apache/maven-doxia/pull/11#discussion_r249863748 ## File path:

[GitHub] [maven-doxia] michael-o commented on issue #10: XHTML5 support for maven-doxia

michael-o commented on issue #10: XHTML5 support for maven-doxia URL: https://github.com/apache/maven-doxia/pull/10#issuecomment-478115447 Please perform the requested changes by applying your patch and I will re-review.

[GitHub] [maven-doxia] minfrin commented on issue #10: XHTML5 support for maven-doxia

minfrin commented on issue #10: XHTML5 support for maven-doxia URL: https://github.com/apache/maven-doxia/pull/10#issuecomment-469780778 > * Why have specialized items replaced with inline? What is the purpose? They were all almost identical variations of what would otherwise have

[GitHub] michael-o commented on a change in pull request #10: XHTML5 support for maven-doxia

michael-o commented on a change in pull request #10: XHTML5 support for maven-doxia URL: https://github.com/apache/maven-doxia/pull/10#discussion_r257521658 ## File path: doxia-core/src/main/java/org/apache/maven/doxia/markup/HtmlMarkup.java ## @@ -25,11 +25,13 @@ /**

[GitHub] michael-o commented on a change in pull request #10: XHTML5 support for maven-doxia

michael-o commented on a change in pull request #10: XHTML5 support for maven-doxia URL: https://github.com/apache/maven-doxia/pull/10#discussion_r257521658 ## File path: doxia-core/src/main/java/org/apache/maven/doxia/markup/HtmlMarkup.java ## @@ -25,11 +25,13 @@ /**

[GitHub] michael-o commented on a change in pull request #10: XHTML5 support for maven-doxia

michael-o commented on a change in pull request #10: XHTML5 support for maven-doxia URL: https://github.com/apache/maven-doxia/pull/10#discussion_r257522186 ## File path: doxia-core/src/main/java/org/apache/maven/doxia/parser/XhtmlBaseParser.java ## @@ -77,9 +77,6 @@

[GitHub] michael-o commented on a change in pull request #10: XHTML5 support for maven-doxia

michael-o commented on a change in pull request #10: XHTML5 support for maven-doxia URL: https://github.com/apache/maven-doxia/pull/10#discussion_r257522353 ## File path: doxia-core/src/test/java/org/apache/maven/doxia/parser/XhtmlBaseParserTest.java ## @@ -555,42

[GitHub] jmones commented on issue #14: [DOXIA-584] Rewrite .md and .markdown links to .html

jmones commented on issue #14: [DOXIA-584] Rewrite .md and .markdown links to .html URL: https://github.com/apache/maven-doxia/pull/14#issuecomment-457496406 I'm improving the solution a little bit more. Will create another pull request.

[GitHub] jmones closed pull request #14: [DOXIA-584] Rewrite .md and .markdown links to .html

jmones closed pull request #14: [DOXIA-584] Rewrite .md and .markdown links to .html URL: https://github.com/apache/maven-doxia/pull/14 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] hboutemy commented on issue #14: [DOXIA-584] Rewrite .md and .markdown links to .html

hboutemy commented on issue #14: [DOXIA-584] Rewrite .md and .markdown links to .html URL: https://github.com/apache/maven-doxia/pull/14#issuecomment-457497988 notice: you can do this by just updating your branch, without closing this PR :) if you can add a little unit test, that

[GitHub] jmones commented on issue #14: [DOXIA-584] Rewrite .md and .markdown links to .html

jmones commented on issue #14: [DOXIA-584] Rewrite .md and .markdown links to .html URL: https://github.com/apache/maven-doxia/pull/14#issuecomment-457609189 Hi Hervé. Now you can look at it. I implemented it as a regex and added some unit tests, as requested. Don't hesitate to request

[GitHub] hboutemy commented on issue #14: [DOXIA-584] Rewrite .md and .markdown links to .html

hboutemy commented on issue #14: [DOXIA-584] Rewrite .md and .markdown links to .html URL: https://github.com/apache/maven-doxia/pull/14#issuecomment-457638840 great work!!! please merge the PR I just sent on your clone for unit tests formatting then squash the commits then I'll

[GitHub] jmones commented on issue #14: [DOXIA-584] Rewrite .md and .markdown links to .html

jmones commented on issue #14: [DOXIA-584] Rewrite .md and .markdown links to .html URL: https://github.com/apache/maven-doxia/pull/14#issuecomment-457651728 Done. I hope it's ok (no much experience on this). Please note that I had to update flexmark dependency in POM, as I needed

[GitHub] jmones opened a new pull request #14: [DOXIA-584] Rewrite .md and .markdown links to .html

jmones opened a new pull request #14: [DOXIA-584] Rewrite .md and .markdown links to .html URL: https://github.com/apache/maven-doxia/pull/14 This implementation supports rewriting the following types of links: (link to a)[a.md] (link to xxx section in a)[a.md#xxx] (link to

[GitHub] hboutemy merged pull request #14: [DOXIA-584] Rewrite .md and .markdown links to .html

hboutemy merged pull request #14: [DOXIA-584] Rewrite .md and .markdown links to .html URL: https://github.com/apache/maven-doxia/pull/14 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] hboutemy commented on issue #14: [DOXIA-584] Rewrite .md and .markdown links to .html

hboutemy commented on issue #14: [DOXIA-584] Rewrite .md and .markdown links to .html URL: https://github.com/apache/maven-doxia/pull/14#issuecomment-457747717 merged, thank you: great job new Flexmark version still requires Java 7, then it's perfect on releasing, there is no precise

[GitHub] [maven-doxia] harawata opened a new pull request #17: [DOXIA-583] Avoid over-escaping table captions

harawata opened a new pull request #17: [DOXIA-583] Avoid over-escaping table captions URL: https://github.com/apache/maven-doxia/pull/17 Please see https://issues.apache.org/jira/browse/DOXIA-583 for details. This is an

[GitHub] [maven-doxia] asfgit closed pull request #17: [DOXIA-583] Avoid over-escaping table captions

asfgit closed pull request #17: [DOXIA-583] Avoid over-escaping table captions URL: https://github.com/apache/maven-doxia/pull/17 This is an automated message from the Apache Git Service. To respond to the message, please

  1   2   3   4   5   6   7   8   9   10   >