chore: added code formatting settings for Positron/Air users #1816
Conversation
Current Aviator status
This pull request is currently open (not queued). How to mergeTo merge this PR, comment
See the real-time status of this PR on the
Aviator webapp.
Use the Aviator Chrome Extension
to see the status of your PR within GitHub.
|
| "editor.defaultFormatter": "positron.positron-r", | ||
| "editor.formatOnType": true |
There was a problem hiding this comment.
These are actually not right. I'd encourage you to use usethis::use_air() in the development version of usethis instead
There was a problem hiding this comment.
i.e. you want these instead, which it sets up for you
https://github.com/tidyverse/dplyr/blob/main/.vscode/settings.json
https://github.com/tidyverse/dplyr/blob/main/.vscode/extensions.json
There was a problem hiding this comment.
You are right and I realize now that I have to rethink this. I turned Air of for igraph on purpose, because it did reformat old styling that was not supposed to be changed on a regular basis. So putting the air.toml in here but turning air off in the settings doesn't make much sense
Fix #1734
the
.vscodefolder contains custom styling rule for positron (might need to see if this needs to change for developers who use vscode, but not positron)air.tomlconfigures air to skip messing withgraph_from_literal.