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.

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

Migrate to slf4j #279

Merged
merged 10 commits into from
May 18, 2023
Merged

Migrate to slf4j #279

merged 10 commits into from
May 18, 2023

Conversation

Nashatyrev
Copy link
Collaborator

@Nashatyrev Nashatyrev commented May 17, 2023

Slf4j is a thinner and more commonly used logging abstraction library

  • Replace log4j-api entries and dependencies with corresponding slf4j analogs
  • Add log4j-slf4j2-impl runtime dependencies for examples, tests and simulator module

Relates to #272

@Nashatyrev Nashatyrev mentioned this pull request May 17, 2023
Copy link
Contributor

@lucassaldanha lucassaldanha left a comment

Choose a reason for hiding this comment

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

LGTM. However, there are some places where we have a bunch of empty spaces between variable name and assignment. Not sure if that was on purpose :)

@@ -27,14 +27,14 @@
* @author Arthur van Hoff, Werner Randelshofer, Pierre Frisch, Daniel Bobbert
*/
public final class DNSIncoming extends DNSMessage {
private static Logger logger = LogManager.getLogger(DNSIncoming.class.getName());
private static Logger logger = LoggerFactory.getLogger(DNSIncoming.class.getName());
Copy link
Contributor

Choose a reason for hiding this comment

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

Why so many spaces before the =? 😅

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

That was just copied sources 'as is'. If at some point we will add Java Spotless that would be fixed :)

@Nashatyrev Nashatyrev merged commit 441e620 into libp2p:v1.0.0 May 18, 2023
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