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

Add logger statements in classes for visibility and debugging #28

Open
3 tasks
avirlrma opened this issue Aug 22, 2023 · 1 comment
Open
3 tasks

Add logger statements in classes for visibility and debugging #28

avirlrma opened this issue Aug 22, 2023 · 1 comment
Labels
sweep Assigns Sweep to an issue or pull request.

Comments

@avirlrma
Copy link
Contributor

avirlrma commented Aug 22, 2023

Details

Use appropriate log levels: trace, debug, info, error.
use @Slf4j annotation from lombok to add logger to classes.
classes: FIUService.java, SetuFIUService.java, RepositoryFacade.java

Checklist
  • fiu-core/src/main/java/com/rupeesense/fi/fiu/FIUService.java

• Add the @Slf4j annotation to import the logger.
• Add appropriate logger statements using the logger variable.

  • fiu-core/src/main/java/com/rupeesense/fi/ext/setu/SetuFIUService.java

• Add the @Slf4j annotation to import the logger.
• Add appropriate logger statements using the logger variable.

  • fiu-dal/src/main/java/com/rupeesense/fi/repo/RepositoryFacade.java

• Add the @Slf4j annotation to import the logger.
• Add appropriate logger statements using the logger variable.

@avirlrma avirlrma added the sweep Assigns Sweep to an issue or pull request. label Aug 22, 2023
@sweep-ai
Copy link

sweep-ai bot commented Aug 22, 2023

66%


❌ Unable to Complete PR

I'm sorry, but it looks like an error has occurred. Try changing the issue description to re-trigger Sweep. If this error persists contact team@sweep.dev.

For bonus GPT-4 tickets, please report this bug on Discord.


Please look at the generated plan. If something looks wrong, please add more details to your issue.

File Path Proposed Changes
fiu-core/src/main/java/com/rupeesense/fi/fiu/FIUService.java Modify fiu-core/src/main/java/com/rupeesense/fi/fiu/FIUService.java with contents:
• Add the @Slf4j annotation to import the logger.
• Add appropriate logger statements using the logger variable.
fiu-core/src/main/java/com/rupeesense/fi/ext/setu/SetuFIUService.java Modify fiu-core/src/main/java/com/rupeesense/fi/ext/setu/SetuFIUService.java with contents:
• Add the @Slf4j annotation to import the logger.
• Add appropriate logger statements using the logger variable.
fiu-dal/src/main/java/com/rupeesense/fi/repo/RepositoryFacade.java Modify fiu-dal/src/main/java/com/rupeesense/fi/repo/RepositoryFacade.java with contents:
• Add the @Slf4j annotation to import the logger.
• Add appropriate logger statements using the logger variable.

🎉 Latest improvements to Sweep:

  • Use Sweep Map to break large issues into smaller sub-issues, perfect for large tasks like "Sweep (map): migrate from React class components to function components"
  • Getting Sweep to format before committing! Check out Sweep Sandbox Configs to set it up.
  • We released a demo of our chunker, where you can find the corresponding blog and code.

💡 To recreate the pull request edit the issue title or description.

@avirlrma avirlrma changed the title Add logger statements in mentioned classes for visibility and debugging Add logger statements in classes for visibility and debugging Aug 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sweep Assigns Sweep to an issue or pull request.
Projects
None yet
Development

No branches or pull requests

1 participant