[GitHub] ant-ivy issue #70: IVY-1321 Make Main.run() public

2018-03-21 Thread supertick
Github user supertick commented on the issue:

https://github.com/apache/ant-ivy/pull/70
  
Looks great !
(had to look up what REPL is - thanks for the heads up)


---

-
To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org
For additional commands, e-mail: dev-h...@ant.apache.org



[GitHub] ant-ivy issue #70: IVY-1321 Make Main.run() public

2018-03-20 Thread twogee
Github user twogee commented on the issue:

https://github.com/apache/ant-ivy/pull/70
  
Thanks for bearing with me 😉please see the revised commit.


---

-
To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org
For additional commands, e-mail: dev-h...@ant.apache.org



[GitHub] ant-ivy issue #70: IVY-1321 Make Main.run() public

2018-03-20 Thread supertick
Github user supertick commented on the issue:

https://github.com/apache/ant-ivy/pull/70
  
I would make a default overload which took only a String[] arg to model 
**exactly** the Main.main input, but all the rest is perfect, including the 
better isCli boolean vs silly cli option

ResolveReport report = Main.run(new String[]{  exact standalone api ... 
})

I would leave the Exception throws signature in the overloaded run method...

Thanks for your consideration, work, and mention ;) !
Cheers !


---

-
To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org
For additional commands, e-mail: dev-h...@ant.apache.org



[GitHub] ant-ivy issue #70: IVY-1321 Make Main.run() public

2018-03-20 Thread twogee
Github user twogee commented on the issue:

https://github.com/apache/ant-ivy/pull/70
  
@supertick if you're fine with this PR, please close #69. I will merge this 
one later.


---

-
To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org
For additional commands, e-mail: dev-h...@ant.apache.org



[GitHub] ant-ivy issue #70: IVY-1321 Make Main.run() public

2018-03-20 Thread twogee
Github user twogee commented on the issue:

https://github.com/apache/ant-ivy/pull/70
  
I'm a bit surprised that StackOverflow does not refer to Apache 
[Confluence](https://cwiki.apache.org/confluence/display/IVY/Programmatic+use+of+Ivy)


---

-
To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org
For additional commands, e-mail: dev-h...@ant.apache.org