GitHub user paulk-asert opened a pull request:
https://github.com/apache/groovy/pull/723
GROOVY-8601: Groovy should have a runner for JUnit5 tests
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/paulk-asert/groovy groovy8601
Alt
Github user paulk-asert closed the pull request at:
https://github.com/apache/groovy/pull/720
---
GitHub user danielsun1106 opened a pull request:
https://github.com/apache/groovy/pull/722
Request YourKit Java Profiler Open Source License
Here is the link to YourKit and search "Open source project license"
in the page:
https://www.yourkit.com/purchase/
**
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/721
---
GitHub user danielsun1106 opened a pull request:
https://github.com/apache/groovy/pull/721
Cache the parameterized info for better performance
Finding the parameterized type is expensive, so it's better to cache it.
You can merge this pull request into a Git repository by running:
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/717
---
GitHub user paulk-asert opened a pull request:
https://github.com/apache/groovy/pull/720
GROOVY-8601: Groovy should have a runner for JUnit5 tests
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/paulk-asert/groovy groovy6801
Alt
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/714
---
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/719
---
Github user danielsun1106 commented on a diff in the pull request:
https://github.com/apache/groovy/pull/717#discussion_r190359768
--- Diff:
src/main/java/org/codehaus/groovy/transform/stc/StaticTypeCheckingVisitor.java
---
@@ -596,6 +597,26 @@ public void visitVariableExpression(
Github user danielsun1106 commented on a diff in the pull request:
https://github.com/apache/groovy/pull/717#discussion_r190359237
--- Diff:
src/main/java/org/codehaus/groovy/transform/stc/StaticTypeCheckingVisitor.java
---
@@ -596,6 +597,26 @@ public void visitVariableExpression(
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/716
---
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/715
---
GitHub user melix opened a pull request:
https://github.com/apache/groovy/pull/719
Optimize generation of Groovydoc
There's a lot to say about Groovydoc, but worst of all, its performance is
terrible. This PR is an attempt to fix part of it.
Before my changes: https://scans
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/718
---
GitHub user danielsun1106 opened a pull request:
https://github.com/apache/groovy/pull/718
Bump ivy to 2.5.0-rc1
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/danielsun1106/groovy bump-ivy-2.5.0-rc1
Alternatively you can revie
GitHub user MeneDev opened a pull request:
https://github.com/apache/groovy/pull/717
Groovy 8562 wrong closure delegation for property access and compile static
Roundup:
The StaticTypeCheckingVisitor would not look consider FieldNode to
potentially delegate to a different F
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/713
---
GitHub user paulk-asert opened a pull request:
https://github.com/apache/groovy/pull/716
GROOVY-8596: SqlGroovyMethods could be moved to avoid split packages â¦
â¦- part (1)
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/paulk-
GitHub user paulk-asert opened a pull request:
https://github.com/apache/groovy/pull/715
GROOVY-8593: DocGenerator could be moved to avoid split packages
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/paulk-asert/groovy groovy85
Github user jwagenleitner commented on the pull request:
https://github.com/apache/groovy/commit/869c365161457c050d5a54c7ff43d73d3263f34e#commitcomment-29066081
thanks!
---
Github user danielsun1106 commented on the pull request:
https://github.com/apache/groovy/commit/b0d79332a3e897e821e62983ba38e5d926fa7cdd#commitcomment-29062398
author is not blank ;-)
---
GitHub user danielsun1106 opened a pull request:
https://github.com/apache/groovy/pull/714
GROOVY-8586: Support Java 9 ARM blocks
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/danielsun1106/groovy GROOVY-8586
Alternatively you
Github user danielsun1106 commented on the pull request:
https://github.com/apache/groovy/commit/869c365161457c050d5a54c7ff43d73d3263f34e#commitcomment-29055018
I've synchronized your changes to
https://github.com/danielsun1106/groovy-parser with your contribution info kept
;-)
GitHub user jwagenleitner opened a pull request:
https://github.com/apache/groovy/pull/713
GROOVY-8590: STC incorrectly infers type of nested method call used iâ¦
â¦n a return stmt
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/712
---
GitHub user jwagenleitner opened a pull request:
https://github.com/apache/groovy/pull/712
GROOVY-8171: Escaped dollar slashy difference between old and parrot â¦
â¦parsers
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jwagenl
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/711
---
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/710
---
GitHub user remkop opened a pull request:
https://github.com/apache/groovy/pull/711
GROOVY-8589 bugfix: should not show error about missing mandatory args when
--help or --version is specified
GROOVY-8589 bugfix: should not show error about missing mandatory args when
--help or --v
GitHub user remkop opened a pull request:
https://github.com/apache/groovy/pull/710
GROOVY-8588 use lowercase -v for --version in groovyc to be consistent with
other Groovy command line tools
GROOVY-8588 use lowercase -v for --version in groovyc to be consistent with
other Groovy c
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/707
---
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/701
---
Github user danielsun1106 commented on the pull request:
https://github.com/apache/groovy/commit/c2cea059fecc3bc4a2c01adc6a30d2a1508bd64b#commitcomment-29031873
author is not blank ð
---
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/709
---
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/702
---
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/704
---
Github user danielsun1106 commented on the pull request:
https://github.com/apache/groovy/commit/c9eea7c30ececdc0a24939fb780c56c27eec798f#commitcomment-29020550
GROOVY-4151 test has been recovered by
https://github.com/apache/groovy/commit/ef2fbe68d25ae7dce57ee2580c975c54e3dbaace
--
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/703
---
GitHub user paulk-asert opened a pull request:
https://github.com/apache/groovy/pull/709
GROOVY-8579: No bytecode level check is done before producing JDK8+ bâ¦
â¦ytecode
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/paulk-ase
Github user blackdrag commented on a diff in the pull request:
https://github.com/apache/groovy/pull/708#discussion_r188869080
--- Diff:
src/main/java/org/codehaus/groovy/transform/stc/StaticTypeCheckingVisitor.java
---
@@ -4101,13 +4101,17 @@ protected static ClassNode
getGroupO
Github user danielsun1106 commented on a diff in the pull request:
https://github.com/apache/groovy/pull/708#discussion_r188837462
--- Diff: src/test/groovy/bugs/Groovy6668Bug.groovy ---
@@ -0,0 +1,65 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
Github user paulk-asert commented on a diff in the pull request:
https://github.com/apache/groovy/pull/708#discussion_r188832656
--- Diff: src/test/groovy/bugs/Groovy6668Bug.groovy ---
@@ -0,0 +1,65 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
Github user danielsun1106 commented on a diff in the pull request:
https://github.com/apache/groovy/pull/708#discussion_r188826337
--- Diff: src/test/groovy/bugs/Groovy6668Bug.groovy ---
@@ -0,0 +1,65 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
Github user danielsun1106 commented on a diff in the pull request:
https://github.com/apache/groovy/pull/708#discussion_r188825988
--- Diff:
src/main/java/org/codehaus/groovy/transform/stc/StaticTypeCheckingVisitor.java
---
@@ -4101,13 +4101,17 @@ protected static ClassNode
getGr
Github user blackdrag commented on a diff in the pull request:
https://github.com/apache/groovy/pull/708#discussion_r188822722
--- Diff: src/test/groovy/bugs/Groovy6668Bug.groovy ---
@@ -0,0 +1,65 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ *
Github user blackdrag commented on a diff in the pull request:
https://github.com/apache/groovy/pull/708#discussion_r188821809
--- Diff:
src/main/java/org/codehaus/groovy/transform/stc/StaticTypeCheckingVisitor.java
---
@@ -4101,13 +4101,17 @@ protected static ClassNode
getGroupO
Github user blackdrag commented on a diff in the pull request:
https://github.com/apache/groovy/pull/708#discussion_r188821671
--- Diff:
src/main/java/org/codehaus/groovy/classgen/asm/sc/StaticInvocationWriter.java
---
@@ -468,7 +469,11 @@ protected void loadArguments(List
argume
GitHub user danielsun1106 opened a pull request:
https://github.com/apache/groovy/pull/708
GROOVY-6668: Static compiler doesn't coerce GString for getAt() call
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/danielsun1106/groovy
GitHub user jwagenleitner opened a pull request:
https://github.com/apache/groovy/pull/707
GROOVY-8509: SC error call to protected method from same package
Because they were closely related, this PR also includes the following:
1. A change to the fix introduced by GROOVY-788
Github user jwagenleitner commented on the pull request:
https://github.com/apache/groovy/commit/b1d1232770aade9672668df4dbc6aa2e2076fa9e#commitcomment-28988363
In subprojects/groovy-sql/src/main/java/groovy/sql/Sql.java:
In subprojects/groovy-sql/src/main/java/groovy/sql/Sql.java
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/706
---
GitHub user danielsun1106 opened a pull request:
https://github.com/apache/groovy/pull/706
GROOVY-8355: Instanceof inference does not work on field assigning
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/danielsun1106/groovy GR
Github user remkop commented on a diff in the pull request:
https://github.com/apache/groovy/pull/704#discussion_r188142573
--- Diff: src/main/groovy/org/codehaus/groovy/tools/GrapeMain.groovy ---
@@ -19,290 +19,323 @@
package org.codehaus.groovy.tools
import groovy.
Github user paulk-asert commented on the pull request:
https://github.com/apache/groovy/commit/62f877abd28c62e3058a383905bb93624ef36130#commitcomment-28966636
Weird, I had to reset again. Seems okay now. I don't know how it keeps
coming back. Maybe I just need to do a fresh reinstall
Github user danielsun1106 commented on the pull request:
https://github.com/apache/groovy/commit/62f877abd28c62e3058a383905bb93624ef36130#commitcomment-28966000
@paulk-asert author is not blank again? o_O
---
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/705
---
Github user danielsun1106 commented on a diff in the pull request:
https://github.com/apache/groovy/pull/705#discussion_r187764707
--- Diff: subprojects/groovy-sql/src/main/java/groovy/sql/Sql.java ---
@@ -3973,7 +3973,7 @@ protected final ResultSet executePreparedQuery(String
sql,
GitHub user danielsun1106 opened a pull request:
https://github.com/apache/groovy/pull/705
GROOVY-7883 Static compiler prefers private constructor over public â¦
â¦if private matches be
You can merge this pull request into a Git repository by running:
$ git pull https://gith
GitHub user remkop opened a pull request:
https://github.com/apache/groovy/pull/704
GROOVY-8577 Migrate org.codehaus.groovy.tools.GrapeMain.groovy to picocli
GROOVY-8577 Migrate org.codehaus.groovy.tools.GrapeMain.groovy to picocli
You can merge this pull request into a Git reposito
Github user paulk-asert commented on the pull request:
https://github.com/apache/groovy/commit/3f650cc681cf1702860a47247bfa728c4c793ef8#commitcomment-28943972
It was changed again! Weird, I've never set that field myself.
---
Github user danielsun1106 commented on the pull request:
https://github.com/apache/groovy/commit/3f650cc681cf1702860a47247bfa728c4c793ef8#commitcomment-28943410
@paulk-asert author field is not blank again? ;-)
---
Github user remkop closed the pull request at:
https://github.com/apache/groovy/pull/700
---
Github user remkop closed the pull request at:
https://github.com/apache/groovy/pull/694
---
GitHub user remkop opened a pull request:
https://github.com/apache/groovy/pull/703
GROOVY-8567 Migrate Groovyc to picocli
Migrate groovyc to picocli.
Rebased and squashed commits. This PR replaces #694.
You can merge this pull request into a Git repository by running:
$ g
Github user remkop closed the pull request at:
https://github.com/apache/groovy/pull/697
---
GitHub user remkop opened a pull request:
https://github.com/apache/groovy/pull/702
GROOVY-8569 GroovyMain picocli migration
Migrate `groovy.ui.GroovyMain` from commons-cli to picocli.
Replaces PR #697
You can merge this pull request into a Git repository by running:
$ git
Github user remkop closed the pull request at:
https://github.com/apache/groovy/pull/696
---
GitHub user remkop opened a pull request:
https://github.com/apache/groovy/pull/701
GROOVY-8520 fixes (attempt 3)
Rebased on master, replaces PR #696
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/remkop/groovy GROOVY-8520-fixes(
GitHub user remkop opened a pull request:
https://github.com/apache/groovy/pull/700
GROOVY-8576 Remove Java2GroovyMain dependency on commons-cli
`org.codehaus.groovy.antlr.java.Java2GroovyMain` can be simplified by
removing its dependency on commons-cli: it does not have any options
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/698
---
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/699
---
GitHub user danielsun1106 opened a pull request:
https://github.com/apache/groovy/pull/699
GROOVY-7204: Static type checking and compilation fail when multiple â¦
â¦generics in use
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
Github user bsideup commented on the pull request:
https://github.com/apache/groovy/commit/33be1e1d51dd0b238ee551eff0c46a2306b7e339#commitcomment-28897694
In
subprojects/groovy-macro/src/main/groovy/org/codehaus/groovy/macro/methods/MacroGroovyMethods.java:
In
subprojects/groovy-
GitHub user ecerulm opened a pull request:
https://github.com/apache/groovy/pull/698
Add documentation for metaclasses
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ecerulm/groovy metaclassdoc2
Alternatively you can review and
GitHub user remkop opened a pull request:
https://github.com/apache/groovy/pull/697
GROOVY-8569 GroovyMain picocli migration
Migrate `groovy.ui.GroovyMain` from commons-cli to picocli.
You can merge this pull request into a Git repository by running:
$ git pull https://github.c
GitHub user remkop opened a pull request:
https://github.com/apache/groovy/pull/696
GROOVY-8520 fixes (attempt 2)
* add CliBuilder.setParser and setFormatter methods that ignore the
specified values and print a warning to stderr to easy the transition for
existing applications
Github user remkop closed the pull request at:
https://github.com/apache/groovy/pull/695
---
GitHub user remkop opened a pull request:
https://github.com/apache/groovy/pull/695
GROOVY-8520 fixes
* add CliBuilder.setParser and setFormatter methods that ignore the
specified values and print a warning to stderr to easy the transition for
existing applications
* bugfix: @U
Github user remkop commented on a diff in the pull request:
https://github.com/apache/groovy/pull/694#discussion_r186292414
--- Diff: src/main/java/org/codehaus/groovy/tools/FileSystemCompiler.java
---
@@ -77,18 +73,6 @@ public void compile(File[] files) throws Exception {
Github user paulk-asert commented on the pull request:
https://github.com/apache/groovy/commit/c80d02b13005efd87eb69ca66958def9becc56e1#commitcomment-28863337
That's it! I was wondering how it got in there. I had checked all my git
config from the cmd line. I don't remember ever filli
Github user danielsun1106 commented on the pull request:
https://github.com/apache/groovy/commit/c80d02b13005efd87eb69ca66958def9becc56e1#commitcomment-28855880
@paulk-asert you can check whether the Author field in the commit dialog of
IntelliJ IDEA is blank.
---
Github user danielsun1106 commented on the pull request:
https://github.com/apache/groovy/commit/c80d02b13005efd87eb69ca66958def9becc56e1#commitcomment-28855805
@paulk-asert I have not contributed the commit...
---
Github user remkop commented on a diff in the pull request:
https://github.com/apache/groovy/pull/694#discussion_r186035996
--- Diff: src/main/java/org/codehaus/groovy/tools/FileSystemCompiler.java
---
@@ -77,18 +73,6 @@ public void compile(File[] files) throws Exception {
Github user remkop commented on a diff in the pull request:
https://github.com/apache/groovy/pull/694#discussion_r186035195
--- Diff: src/main/java/org/codehaus/groovy/tools/FileSystemCompiler.java
---
@@ -77,18 +73,6 @@ public void compile(File[] files) throws Exception {
Github user paulk-asert commented on a diff in the pull request:
https://github.com/apache/groovy/pull/694#discussion_r186015829
--- Diff: src/main/java/org/codehaus/groovy/tools/FileSystemCompiler.java
---
@@ -77,18 +73,6 @@ public void compile(File[] files) throws Exception {
Github user paulk-asert commented on a diff in the pull request:
https://github.com/apache/groovy/pull/694#discussion_r186015291
--- Diff: src/main/java/org/codehaus/groovy/tools/FileSystemCompiler.java
---
@@ -77,18 +73,6 @@ public void compile(File[] files) throws Exception {
GitHub user remkop opened a pull request:
https://github.com/apache/groovy/pull/694
GROOVY-8567 Migrate Groovyc to picocli
GROOVY-8567 Migrate Groovyc to picocli
See comments in the JIRA ticket.
You can merge this pull request into a Git repository by running:
$ git pu
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/693
---
Github user ecerulm commented on a diff in the pull request:
https://github.com/apache/groovy/pull/693#discussion_r185943651
--- Diff: src/main/groovy/groovy/lang/MetaClassImpl.java ---
@@ -945,6 +945,15 @@ private Object invokeMissingMethod(Object instance,
String methodName, Obje
Github user danielsun1106 commented on a diff in the pull request:
https://github.com/apache/groovy/pull/693#discussion_r185901767
--- Diff: src/main/groovy/groovy/lang/MetaClassImpl.java ---
@@ -945,6 +945,15 @@ private Object invokeMissingMethod(Object instance,
String methodName
GitHub user ecerulm opened a pull request:
https://github.com/apache/groovy/pull/693
Allow methodMissing/propertyMissing to be defined through category
Resolves GROOVY-3867
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ecerulm/g
Github user paulk-asert commented on the pull request:
https://github.com/apache/groovy/commit/aa1ad8fe2ab2c37a0b95215e692e73bdc37297f7#commitcomment-28813835
It was me - I am not sure how your name got there. I'll have to check
whether my git config is messed up.
---
Github user danielsun1106 commented on the pull request:
https://github.com/apache/groovy/commit/aa1ad8fe2ab2c37a0b95215e692e73bdc37297f7#commitcomment-28806379
weird... I have not contributed such a change, but the author is me...
---
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/692
---
GitHub user remkop opened a pull request:
https://github.com/apache/groovy/pull/692
GROOVY-8520 CliBuilder groovydoc improvements
Groovy-doc improvements for CliBuilder.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/remkop/groov
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/688
---
GitHub user danielsun1106 opened a pull request:
https://github.com/apache/groovy/pull/691
Support java-like array
## Background
In order to improve the compatibility with Java(Copy & Paste), Java array
is one of features widely applied in Java projects. The PR will make Groovy
Github user paulk-asert commented on the pull request:
https://github.com/apache/groovy/commit/5443e87882f9b88169876f6d043ed54b5ae9023b#commitcomment-28711056
In src/main/java/org/codehaus/groovy/control/CompilerConfiguration.java:
In src/main/java/org/codehaus/groovy/control/Compi
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/690
---
301 - 400 of 2069 matches
Mail list logo