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.

Sign up for GitHub

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

Jump to bottom

Remove line breaks between `}` and `else` #793

Merged

lorenzwalthert merged 3 commits into r-lib:master from lorenzwalthert:issue-792

May 6, 2021

Collaborator

lorenzwalthert commented May 5, 2021 •

edited

Closes #792.

Here is how the current PR would change benchmark results when merged into master:

cache_applying: 0.11 -> 0.11 (0.8%)
cache_recording: 1.54 -> 1.55 (0.4%)
without_cache: 3.98 -> 3.99 (0.4%)

lorenzwalthert added 2 commits

May 6, 2021 01:01


          remove line break before else if possible

fb288a0


          add news bullet

b6d3c41

lorenzwalthert force-pushed the issue-792 branch from 47d4604 to b6d3c41 Compare

May 5, 2021 23:06


          adapt out test

2ca8c26

lorenzwalthert merged commit 2b10dfc into r-lib:master

lorenzwalthert deleted the issue-792 branch

May 6, 2021 07:36

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