Skip to content

fix: update code to pass latest test suite - #174

Open
dwalluck wants to merge 8 commits into
package-url:masterfrom
dwalluck:update-test-suite
Open

fix: update code to pass latest test suite#174
dwalluck wants to merge 8 commits into
package-url:masterfrom
dwalluck:update-test-suite

Conversation

@dwalluck

Copy link
Copy Markdown
Contributor

No description provided.

Comment threadsrc/test/resources/test-suite-data.json
Comment threadsrc/test/resources/test-suite-data.json
Comment threadsrc/test/resources/test-suite-data-java.json Outdated
@jeremylong

Copy link
Copy Markdown
Collaborator

@dwalluck mind fixing the merge conflicts?

@dwalluck

Copy link
Copy Markdown
ContributorAuthor

Needs #205 first.

@jeremylong

Copy link
Copy Markdown
Collaborator

There is one more merge conflict that needs to be resolved.

@dwalluck

Copy link
Copy Markdown
ContributorAuthor

This really needs #176 first, as this pull request basically contains a version of that pull request. I am not sure why I submitted the one required first after this one.

@dwalluckdwalluck changed the title Update for latest test suitefix: update code to latest test suiteMar 18, 2025
@dwalluckdwalluck changed the title fix: update code to latest test suitefix: update code to pass latest test suiteMar 18, 2025
@ppkarwasz

Copy link
Copy Markdown
Contributor

Now that Spotless is there, there will be some conflicts. This should help:

git checkout origin/master -- pom.xml
mvn spotless:apply

Comment threadsrc/test/java/com/github/packageurl/PackageURLTest.java
Sign up for freeto 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.

3 participants

@dwalluck@jeremylong@ppkarwasz