Skip to content
This repository has been archived by the owner on Sep 22, 2024. It is now read-only.

Add Codespace .NET dev environment for LodeRunner repo #266

Merged
merged 6 commits into from
Jul 7, 2022

Conversation

rcaseymsft
Copy link
Contributor

Type of PR

  • Documentation changes
  • Code changes
  • Test changes
  • CI-CD changes
  • GitHub Template changes
  • Container configuration

PR Checklist

  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • My code follows the code style of this project.
  • I ran lint checks locally prior to submission.
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Purpose of PR

  • Update Codespace C# development environment for Burst metrics
    • Omnisharp settings update (global Roslyn settings and vscode specific)*
    • VSCode editorconfig support*
    • Added build tasks for LodeRunner.API, LodeRunner
    • Added launch settings for LodeRunner and LodeRunner.API
    • IDE improvements (sort using)
    • Added dotnet test explorer

Does this introduce a breaking change

  • YES
  • NO

Validation

  • Unit tests updated and ran successfully
  • Update documentation or issue referenced above

Issues Closed or Referenced

@rcaseymsft rcaseymsft added DevX Easy for developer to understand/deploy size: M 2-3 days labels Jul 5, 2022
@rcaseymsft rcaseymsft self-assigned this Jul 5, 2022
@jofultz jofultz removed the size: M 2-3 days label Jul 5, 2022
Copy link
Contributor

@AAkindele AAkindele left a comment

Choose a reason for hiding this comment

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

I was able to successfully test the new features that were described in the README.

@rcaseymsft rcaseymsft merged commit a8f2b58 into main Jul 7, 2022
@rcaseymsft rcaseymsft deleted the rcaseymsft/cs-dev-env branch July 7, 2022 13:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
DevX Easy for developer to understand/deploy
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants