Skip to content

Remove ConsoleLauncher configuration in Ant starter (#98)#114

Open
seachicken wants to merge 4 commits intojunit-team:mainfrom
seachicken:issue98
Open

Remove ConsoleLauncher configuration in Ant starter (#98)#114
seachicken wants to merge 4 commits intojunit-team:mainfrom
seachicken:issue98

Conversation

@seachicken
Copy link

@seachicken seachicken commented Oct 2, 2019

Overview

Fixes #98
Fixes #115


I hereby agree to the terms of the JUnit Contributor License Agreement.

@sbrannen sbrannen changed the title Remove ConsoleLauncher configuration (#98) Remove ConsoleLauncher configuration in Ant starter (#98) Oct 2, 2019
@sbrannen sbrannen dismissed their stale review October 2, 2019 16:10

Let's push the use of Ivy to a separate issue

@seachicken seachicken requested a review from sbrannen October 3, 2019 14:42
Copy link
Member

@sormuras sormuras left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sbrannen May I consider your requested changes as resolved and merge this PR?

@marcphilipp
Copy link
Member

With two approvals and since I think his requested changes have been resolved, I think we can overrule @sbrannen this once.

@marcphilipp marcphilipp changed the base branch from master to main June 19, 2020 18:29
@marcphilipp
Copy link
Member

Team decision: Keep using plain Ant

@marcphilipp
Copy link
Member

Actually, we missed the main point about moving away from using the ConsoleLauncher.

@marcphilipp
Copy link
Member

@sormuras will take it from here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use Apache Ivy in Ant examples Remove ConsoleLauncher configuration in Ant starter

4 participants