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

WatchService : Anonymous new Runnable() can be replaced with lambda #1780

Open
2 tasks
rohanKanojia opened this issue Apr 13, 2024 · 4 comments
Open
2 tasks
Assignees

Comments

@rohanKanojia
Copy link
Member

Description

This anonymous class can be replaced with a lambda:

return new Runnable() {
@Override
public void run() {
List<AssemblyFiles.Entry> entries = files.getUpdatedEntriesAndRefresh();

Acceptance Criteria

  • Replace anonymous class with lambda
  • Use isEmpty() in place of size() > 0 check
@rohanKanojia
Copy link
Member Author

@mdxabu Could you please review this one? Is it clear enough for you to pick?

@mdxabu
Copy link
Contributor

mdxabu commented Apr 14, 2024

@mdxabu Could you please review this one? Is it clear enough for you to pick?
yes, I can pick this

@alpha951
Copy link
Contributor

If a PR has not been raised yet for this issue, could you please assign it to me? I've submitted a PR for the previous assignment #1782

@mdxabu
Copy link
Contributor

mdxabu commented Apr 15, 2024

If a PR has not been raised yet for this issue, could you please assign it to me? I've submitted a PR for the previous assignment #1782

Sorry, This issue was assigned for me, I am working on it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants