Skip to content

reformat code based on air#7

Merged
mhpob merged 1 commit intomainfrom
add-air
Sep 10, 2025
Merged

reformat code based on air#7
mhpob merged 1 commit intomainfrom
add-air

Conversation

@benjaminhlina
Copy link
Collaborator

I accidentally pushed the air.toml file that usethis::use_air() creates to main. I ran air on the whole repo and these changes are formatting changes that air did. We should/I know you have implemented a GH styler and I know there is one for air as well.

No actual changes to the code just reformatting

@benjaminhlina benjaminhlina self-assigned this Sep 10, 2025
@benjaminhlina benjaminhlina requested a review from mhpob September 10, 2025 02:44
@mhpob
Copy link
Collaborator

mhpob commented Sep 10, 2025

We don't have the style action on this repo, so no worries there. I'm happy using air and just resetting the code base accordingly as you've done.

@mhpob mhpob merged commit e7174bb into main Sep 10, 2025
6 checks passed
@benjaminhlina benjaminhlina deleted the add-air branch September 10, 2025 12:38
@benjaminhlina
Copy link
Collaborator Author

Awesome!

@mhpob
Copy link
Collaborator

mhpob commented Sep 10, 2025

FYI, I just pushed an edit directly to main -- we had forgotten to put air.toml into .Rbuildignore. I figured we could skip the PR for the hotfix

@benjaminhlina
Copy link
Collaborator Author

Damn! I thought I had gotten air.toml into .Rbuildignore...thanks for the fix

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.

2 participants