Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Improve VSCode configuration #18026

Merged
merged 1 commit into from
Aug 13, 2024
Merged

Improve VSCode configuration #18026

merged 1 commit into from
Aug 13, 2024

Conversation

MikeMcQuaid
Copy link
Member

  • Update outdated and add new recommended extensions
  • Exclude some vendored code from search results by default
  • Add more VSCode editor settings that are consistent with our linting
  • Provide configuration for coverage visualization in VSCode but disable it by default
  • Add SimpleCov coverage for eval and ensure that we always default to line coverage in case upstream defaults ever change

@MikeMcQuaid MikeMcQuaid requested a review from Rylan12 August 12, 2024 16:30
- Update outdated and add new recommended extensions
- Exclude some vendored code from search results by default
- Add more VSCode editor settings that are consistent with our linting
- Provide configuration for coverage visualization in VSCode but disable
  it by default
- Add SimpleCov coverage for `eval` and ensure that we always default to
  line coverage in case upstream defaults ever change
Copy link
Member

@Rylan12 Rylan12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All looks great to me, thanks! I haven't had a chance to test this in VSCode, but I should be able to do that tomorrow. No need to wait to merge, though

@MikeMcQuaid MikeMcQuaid merged commit aad48a0 into master Aug 13, 2024
24 checks passed
@MikeMcQuaid MikeMcQuaid deleted the vscode_tweaks branch August 13, 2024 07:13
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants