Github user PascalSchumacher commented on a diff in the pull request:
https://github.com/apache/groovy/pull/808#discussion_r224093386
--- Diff: src/spec/doc/core-metaprogramming.adoc ---
@@ -1985,7 +1985,19 @@ _protectedCacheSize>0_ would create an unlimited
cache with s
Github user PascalSchumacher commented on the pull request:
https://github.com/apache/groovy/commit/a7ebe87b8cb4ff7ce220a13af96788118d316006#commitcomment-30427718
In subprojects/groovy-test-junit5/build.gradle:
In subprojects/groovy-test-junit5/build.gradle on line 28:
Great
Github user PascalSchumacher commented on the pull request:
https://github.com/apache/groovy/commit/a7ebe87b8cb4ff7ce220a13af96788118d316006#commitcomment-30426446
In subprojects/groovy-test-junit5/build.gradle:
In subprojects/groovy-test-junit5/build.gradle on line 28
Github user PascalSchumacher commented on the pull request:
https://github.com/apache/groovy/commit/b02d2f57c09ba6ba46553ed77aff191ec9ab274e#commitcomment-22219191
In
subprojects/parser-antlr4/src/main/java/org/apache/groovy/parser/antlr4/util/StringUtils.java:
In
subprojects
Github user PascalSchumacher commented on the pull request:
https://github.com/apache/groovy/commit/b02d2f57c09ba6ba46553ed77aff191ec9ab274e#commitcomment-22219131
In
subprojects/parser-antlr4/src/main/java/org/apache/groovy/parser/antlr4/util/StringUtils.java:
In
subprojects
Github user PascalSchumacher commented on the pull request:
https://github.com/apache/groovy/commit/0fb89906aa587920d11fae063bba1d1f8fe26254#commitcomment-21966439
@danielsun1106 So is it a bug in the new groovy parser?
---
If your project is set up for it, you can reply to this
Github user PascalSchumacher commented on the pull request:
https://github.com/apache/groovy/commit/0fb89906aa587920d11fae063bba1d1f8fe26254#commitcomment-21962912
test fails on travis, see:
https://travis-ci.org/apache/groovy/builds/227402483?utm_source=github_status&utm_me
GitHub user PascalSchumacher opened a pull request:
https://github.com/apache/groovy/pull/527
remove skipping of tests for builds.apache.org as we no longer use it
even groovy decides to use builds.apache.org again, these workarounds may
not be necessary today
You can merge this
GitHub user PascalSchumacher opened a pull request:
https://github.com/apache/groovy/pull/526
cleanup now that jdk7 is baseline
for master and the GROOVY_2_5_X branch
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
Github user PascalSchumacher closed the pull request at:
https://github.com/apache/groovy/pull/132
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user PascalSchumacher commented on a diff in the pull request:
https://github.com/apache/groovy/pull/442#discussion_r82509499
--- Diff:
src/spec/test-resources/META-INF/services/org.codehaus.groovy.runtime.ExtensionModule
---
@@ -17,5 +17,5 @@
# IT IS A WORKAROUND FOR
Github user PascalSchumacher commented on the pull request:
https://github.com/apache/groovy/commit/e21d6226b9a4cc411fb0e7d24cf7b6023edc9002#commitcomment-18124607
Nice :+1: I thought nobody would ever fix these tests.
---
If your project is set up for it, you can reply to this
Github user PascalSchumacher commented on a diff in the pull request:
https://github.com/apache/groovy/pull/328#discussion_r62416234
--- Diff:
src/test/org/codehaus/groovy/classgen/ClassCompletionVerifierTest.java ---
@@ -73,13 +73,22 @@
"Method
Github user PascalSchumacher commented on the pull request:
https://github.com/apache/groovy/commit/a0aed6c6e523d176ebd480075d54aa9a3e19b472#commitcomment-16546079
In src/main/org/codehaus/groovy/ast/PropertyNode.java:
In src/main/org/codehaus/groovy/ast/PropertyNode.java on line
Github user PascalSchumacher commented on the pull request:
https://github.com/apache/groovy/commit/a0aed6c6e523d176ebd480075d54aa9a3e19b472#commitcomment-16545920
In src/main/org/codehaus/groovy/ast/PropertyNode.java:
In src/main/org/codehaus/groovy/ast/PropertyNode.java on line
Github user PascalSchumacher commented on the pull request:
https://github.com/apache/groovy/commit/a0aed6c6e523d176ebd480075d54aa9a3e19b472#commitcomment-16545765
In src/main/org/codehaus/groovy/ast/PropertyNode.java:
In src/main/org/codehaus/groovy/ast/PropertyNode.java on line
Github user PascalSchumacher commented on the pull request:
https://github.com/apache/groovy/commit/a0aed6c6e523d176ebd480075d54aa9a3e19b472#commitcomment-16545493
In src/main/org/codehaus/groovy/ast/PropertyNode.java:
In src/main/org/codehaus/groovy/ast/PropertyNode.java on line
Github user PascalSchumacher commented on a diff in the pull request:
https://github.com/apache/groovy/pull/255#discussion_r55146131
--- Diff:
subprojects/groovy-xml/src/test/groovy/groovy/xml/jaxb/JaxbGroovyMethodsTest.groovy
---
@@ -0,0 +1,41 @@
+/*
+ * Licensed to the
Github user PascalSchumacher commented on a diff in the pull request:
https://github.com/apache/groovy/pull/255#discussion_r55146104
--- Diff:
subprojects/groovy-xml/src/main/groovy/groovy/xml/jaxb/JaxbGroovyMethods.groovy
---
@@ -0,0 +1,54 @@
+/*
+ * Licensed to the
GitHub user PascalSchumacher opened a pull request:
https://github.com/apache/groovy/pull/284
GROOVY-7640:AST Builder should include superclass properties
The code is really ugly, but this is caused by a method with is in
`BuilderASTTransformation` but should have been in
Github user PascalSchumacher commented on the pull request:
https://github.com/apache/groovy/commit/53ac431f807f629ad49bfb91a511ff95b42828ca#commitcomment-15849845
:+1:
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user PascalSchumacher commented on the pull request:
https://github.com/apache/groovy/commit/efee13bd34862dbcbdebf79254043664861eb2df#commitcomment-15849844
:+1:
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user PascalSchumacher commented on the pull request:
https://github.com/apache/groovy/commit/f7be98a084cf3d6b8a82cae05fa1ec32f7ed7029#commitcomment-15849840
:+1:
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user PascalSchumacher commented on the pull request:
https://github.com/apache/groovy/commit/2e7c69dc4d185dc790b7d15857235458ef3eb7a6#commitcomment-15660685
Thanks! Sorry for not checking before merging the pull request.
---
If your project is set up for it, you can reply to
Github user PascalSchumacher commented on the pull request:
https://github.com/apache/groovy/commit/606891d85bedbda8139882f41eef531f5752d99f#commitcomment-15434092
I guess this could/should also be merged into 2.4.x What do you think?
---
If your project is set up for it, you can
GitHub user PascalSchumacher opened a pull request:
https://github.com/apache/groovy/pull/216
GROOVY-7074: DelegatingScript does not work with Builder as delegate
Patch supplied by Jochen Kemnade
You can merge this pull request into a Git repository by running:
$ git pull
26 matches
Mail list logo