Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

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

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Fix: Remove duplicate commons-io dependency in pom.xml#259

Open
vitinh0z wants to merge 2 commits intospotify:masterspotify/github-java-client:masterfrom
vitinh0z:fix/pom-duplicatevitinh0z/github-java-client:fix/pom-duplicateCopy head branch name to clipboard
Open

Fix: Remove duplicate commons-io dependency in pom.xml#259
vitinh0z wants to merge 2 commits intospotify:masterspotify/github-java-client:masterfrom
vitinh0z:fix/pom-duplicatevitinh0z/github-java-client:fix/pom-duplicateCopy head branch name to clipboard

Conversation

@vitinh0z
Copy link

Description

This PR fixes a build failure caused by a duplicate declaration of the commons-io dependency in pom.xml.

The Problem

The project was failing to build with a ProjectBuildingException because commons-io (version 2.14.0) was declared twice in the dependencies list.

The Fix

I removed the duplicate entry.

Verification

I ran mvn clean install -DskipTests locally and confirmed that the build now succeeds.

Copilot AI review requested due to automatic review settings December 16, 2025 14:21
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes a build failure by removing a duplicate commons-io dependency declaration in the Maven POM file. The duplicate entry was causing a ProjectBuildingException during the build process.

Key Changes:

  • Removed duplicate commons-io dependency (version 2.14.0) from the dependencies section
  • Added root="true" attribute to the project tag (likely unintentional)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

pom.xml Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.