Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Polish maven test for convenient debug#134

Merged
wu-sheng merged 1 commit intomainfrom
debug
Sep 3, 2022
Merged

Polish maven test for convenient debug#134
wu-sheng merged 1 commit intomainfrom
debug

Conversation

@kezhenxu94
Copy link
Member

Now we can debug the identification in the test, which makes it simple to debug why we failed to identify the license of a dependency

@kezhenxu94 kezhenxu94 added the enhancement New feature or request label Sep 3, 2022
@kezhenxu94 kezhenxu94 added this to the 0.5.0 milestone Sep 3, 2022
@wu-sheng
Copy link
Member

wu-sheng commented Sep 3, 2022

I can't follow the PR description. What does this change mean?

Comment on lines +135 to +139
<dependency>
<groupId>com.fasterxml.jackson.datatype</groupId>
<artifactId>jackson-datatype-jsr310</artifactId>
<version>2.13.3</version>
</dependency>
Copy link
Member Author

Choose a reason for hiding this comment

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

I can't follow the PR description. What does this change mean?

Previously when the tool failed to identify a dependency's license, we have to create a demo maven project and run the tool in debug mode, now we can just put the dependency in the test codes and debug the test, like this.

@wu-sheng wu-sheng merged commit c27b7b3 into main Sep 3, 2022
@wu-sheng wu-sheng deleted the debug branch September 3, 2022 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants