-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add Visual Studio Code workspace configuration #117
Conversation
primer-app.code-workspace
Outdated
"arrterian.nix-env-selector", | ||
"b4dm4n.nixpkgs-fmt", | ||
"bbenoist.nix", | ||
"dbaeumer.vscode-eslint", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Somehow I get a lot of eslint
warnings which don't appear on CI. I'm still investigating that.
primer-app.code-workspace
Outdated
"files.insertFinalNewline": true, | ||
"files.trimFinalNewlines": true, | ||
"files.trimTrailingWhitespace": true, | ||
"nixEnvSelector.nixFile": "${workspaceRoot}/shell.nix", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This isn't ideal, but there's no flake support yet: arrterian/nix-env-selector#53.
And we're rarely needing to reload the editor in practice, so the fact that it takes a few seconds longer isn't a huge issue.
I'm fine with this, so just go ahead and commit it when you're finished with the config. |
We were sticking to VScode's built-in formatter, since I thought `primer-app.code-workspace` was the only "jsonc" file in the project. But it turns out that the `tsconfig.json` files also allow comments, and we use Prettier on those.
As was configured in the previous commit.
Prettier handles this for us on all source files anyway, and for other files we needn't care (there is currently trailing whitespace in the README, at least).
bors merge |
Build succeeded: |
I've argued against editor-specific configuration before, but this would be a useful way for me to be able to share settings with @annedino4...