Weed out ITs by cstamas · Pull Request #45 · apache/maven-plugin-tools (original) (raw)

Appearance settings

- AI CODE CREATION
  * GitHub CopilotWrite better code with AI
  * GitHub Copilot appDirect agents from issue to merge
  * MCP RegistryNewIntegrate external tools
- DEVELOPER WORKFLOWS
  * ActionsAutomate any workflow
  * CodespacesInstant dev environments
  * IssuesPlan and track work
  * Code ReviewManage code changes
- APPLICATION SECURITY
  * GitHub Advanced SecurityFind and fix vulnerabilities
  * Code securitySecure your code as you build
  * Secret protectionStop leaks before they start
- EXPLORE
  * Why GitHub
  * Documentation
  * Blog
  * Changelog
  * Marketplace

View all features

- BY COMPANY SIZE
  * Enterprises
  * Small and medium teams
  * Startups
  * Nonprofits
- BY USE CASE
  * App Modernization
  * DevSecOps
  * DevOps
  * CI/CD
  * View all use cases
- BY INDUSTRY
  * Healthcare
  * Financial services
  * Manufacturing
  * Government
  * View all industries

View all solutions

- EXPLORE BY TOPIC
  * AI
  * Software Development
  * DevOps
  * Security
  * View all topics
- EXPLORE BY TYPE
  * Customer stories
  * Events & webinars
  * Ebooks & reports
  * Business insights
  * GitHub Skills
- SUPPORT & SERVICES
  * Documentation
  * Customer support
  * Community forum
  * Trust center
  * Partners

View all resources

- COMMUNITY
  * GitHub SponsorsFund open source developers
- PROGRAMS
  * Security Lab
  * Maintainer Community
  * Accelerator
  * GitHub Stars
  * Archive Program
- REPOSITORIES
  * Topics
  * Trending
  * Collections
- ENTERPRISE SOLUTIONS
  * Enterprise platformAI-powered developer platform
- AVAILABLE ADD-ONS
  * GitHub Advanced SecurityEnterprise-grade security features
  * Copilot for BusinessEnterprise-grade AI features
  * Premium SupportEnterprise-grade 24/7 support
Pricing

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

apache / maven-plugin-tools Public

Notifications You must be signed in to change notification settings
Fork70
Star 63
Code
Issues 41
Pull requests 6
Actions
Security and quality
Insights

Additional navigation options

Merged

cstamas

merged 1 commit into

masterfrom

weed-out-its

Oct 19, 2021

Conversation Commits (1)Checks Files changed

Merged

Weed out ITs#45

cstamas

merged 1 commit into

masterfrom

weed-out-its

Conversation

@cstamas cstamas commented

Oct 18, 2021

Copy link Copy Markdown

Member

maven-plugin-api 2.0? Really?

`[Weed out ITs](/apache/maven-plugin-tools/pull/45/commits/e29e8519f6f644043331a22f90cd90e629cc8e05 "Weed out ITs

maven-plugin-api 2.0? Really?") `

[e29e851](/apache/maven-plugin-tools/pull/45/commits/e29e8519f6f644043331a22f90cd90e629cc8e05)

maven-plugin-api 2.0? Really?

cstamas requested review fromgnodet and michael-o

October 18, 2021 08:14

cstamas self-assigned this

Oct 18, 2021

michael-o commented

Oct 18, 2021

Copy link Copy Markdown

Member

maven-plugin-api 2.0? Really?

Because we can!

gnodet approved these changes Oct 19, 2021

View reviewed changes

@gnodet gnodet left a comment

Copy link Copy Markdown

Contributor

Choose a reason for hiding this comment

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

Everything looks good, ITs are passing.

cstamas merged commit 960d150 into master

Oct 19, 2021

cstamas deleted the weed-out-its branch

October 19, 2021 07:49

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})