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

stylerignore should be respected for alignemnt detection #850

Merged
merged 1 commit into from
Oct 8, 2021

Conversation

lorenzwalthert
Copy link
Collaborator

No description provided.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 8, 2021

This is how benchmark results would change (along with a 95% confidence interval in relative change) if fc259ab is merged into master:

  •   :ballot_box_with_check:cache_applying: 24.7ms -> 24.8ms [-1.19%, +2.39%]
  •   :ballot_box_with_check:cache_recording: 1.04s -> 1.03s [-3.12%, +0.53%]
  •   :ballot_box_with_check:without_cache: 2.68s -> 2.68s [-1.81%, +2.02%]

Further explanation regarding interpretation and methodology can be found in the documentation.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 8, 2021

This is how benchmark results would change (along with a 95% confidence interval in relative change) if 6f98f45 is merged into master:

  •   :ballot_box_with_check:cache_applying: 37.8ms -> 38.2ms [-2.33%, +4.06%]
  •   :ballot_box_with_check:cache_recording: 1.76s -> 1.73s [-3.63%, +0.79%]
  •   :ballot_box_with_check:without_cache: 4.37s -> 4.35s [-2.64%, +1.84%]

Further explanation regarding interpretation and methodology can be found in the documentation.

@lorenzwalthert lorenzwalthert merged commit 58c8ca5 into r-lib:master Oct 8, 2021
@lorenzwalthert lorenzwalthert deleted the stylerignore-alignment branch October 8, 2021 20:27
@lorenzwalthert lorenzwalthert mentioned this pull request Mar 12, 2022
15 tasks
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.

None yet

1 participant