Skip to content

Don't sync on #tap#82

Open
tjwallace wants to merge 1 commit intoexAspArk:mainfrom
tjwallace:ignore_tap
Open

Don't sync on #tap#82
tjwallace wants to merge 1 commit intoexAspArk:mainfrom
tjwallace:ignore_tap

Conversation

@tjwallace
Copy link

@tjwallace tjwallace commented May 9, 2022

Sometimes #tap is used just for code style and the value is never checked, so don't sync the value unnecessarily.

(Contrived) Example:

lazy_fetch(1).tap { logger.info('lazy_fetch complete') }

Sometimes #tap is used just for code style and the value is never
checked, so don't sync the value.
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.

1 participant

Comments