Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Drop Invokable interface #291

Merged
merged 1 commit into from
May 7, 2022
Merged

Drop Invokable interface #291

merged 1 commit into from
May 7, 2022

Conversation

pzygielo
Copy link
Contributor

@pzygielo pzygielo commented May 7, 2022 •
edited
Loading

At least since

the target is 8, and per comment

exec-maven-plugin/src/main/java/org/codehaus/mojo/exec/Invokable.java

Lines 23 to 24 in eb2f776

* A simple Java 7 pseudo-class based on the Java 8 {@code Consumer} class. Can and should be
* deleted once this project moves to a minimum execution environment of Java 8+.

@pzygielo pzygielo marked this pull request as ready for review May 7, 2022 12:22
@pzygielo

This comment was marked as outdated.

Sign in to view
@pzygielo pzygielo marked this pull request as draft May 7, 2022 12:48
@pzygielo

This comment was marked as resolved.

Sign in to view
@pzygielo pzygielo marked this pull request as ready for review May 7, 2022 18:00
@slachiewicz slachiewicz merged commit f75c252 into mojohaus:master May 7, 2022
@pzygielo pzygielo deleted the invokable branch May 8, 2022 06:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
maintenance
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@pzygielo @olamy @slachiewicz