Disable 'Just My Code' debugging in vscode #52
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
We use HotChocolate in an uncommon way, this means documentation is often of little help. So instead when we run into issues we need to read the source code. Finding the right piece of code to read can be made a lot easier if the debugger can show us what code is being executed. By disabling 'Just My Code' the debugger will allow us to debug code from the HotChocolate library that's being executed.
JelteF
requested review from
Aniruddh25,
jarupatj,
mbhaskar,
moderakh and
seantleonard
as code owners
Aniruddh25
approved these changes
Oct 8, 2021
moderakh
approved these changes
Oct 8, 2021
abhishekkumams
added a commit
that referenced
this pull request
Jul 25, 2022
* replace name to config * symplifying update logic * adding validations * added more tests * resolved comments * adding test description * updating file names for Add and Update EntityTests * adding comments and tests * adding class comments * adding class comments * adding more tests and comments * updated accessebiity of some methods * updating tests * adding TODO for policy support * updating few comments * updated function summary * minor change * fixed comments * refactoring changes
abhishekkumams
added a commit
that referenced
this pull request
Aug 3, 2022
* Initial commit * added first design proposal * CODE_OF_CONDUCT.md committed * LICENSE committed * README.md updated to template * SECURITY.md committed * SUPPORT.md committed * new console app * initial config generator * updated implementation proposal * Update README.md * adding .gitignore * updating gitignore * removing compiled files * added support for command-add * updating gitignore * cleaning files and adding todos * Update README.md * updated file path for config generation * using classes of hawaii-gql * adding design_doc and added update command * removing temp file * updating readme * updating readme * removing temp files * removing temp files * updating README * removing temp files * fixing the update command * removing temp files * Added relationship compatibility * updated ReadMe * removing temp files * updating gitignore * update README.md (#5) * Update README.md (#8) * Update README.md * Update README.md * Fix for connection-string issue (#9) * fix connection-string issue * removing temp files * removing resolver config as it is specific to cosmosDB * updating .gitignore * updating .gitignore * Dev/abhishekkuma/code refactoring (#16) * fix connection-string issue * removing temp files * Use Action class from Runtime Config * removing temp files * minor fix * code refactoring * added summary for all the methods * making mappings.field optional (#13) * Update default host global settings (#19) * updated default host global settings * updating default value of Cors.Origin * fixing spelling errors * Add support for linking object in relationship (#22) * adding support for linking object in relationship * updating validation checks * fixing grammatical errors * Add CODEOWNERS file (#24) * added CODEOWNERS file * updating names * adding Davide as code owner * Update version to 0.0.2 (#25) * updating version to 0.0.2 * fix indentation * Add commands in help window (#27) * added commands in help window * updated help screen * updated help screen * fixing typo * Update README.md (#26) * Update README.md Updating Install Information in Readme. * Update README.md Co-authored-by: Aniruddh Munde <[email protected]> Co-authored-by: Aniruddh Munde <[email protected]> * Updated Typos (#30) * Quick fix for typo (#31) * Add editorconfig to auto format code (#35) * Add editorconfig * Fix formatting warning * Fix more warning * use is null * Update CommandLineOptions property to start with capital letter. * Add skeleton tests project (#36) * Add tests project * Update gitignore * fixing update command for permissions (#34) * Refactoring commandline handling options. (#37) **What are the changes** This change refactors the code to use Verb attribute https://github.com/commandlineparser/commandline/wiki/Verbs to link command line options to command line operation. The main change are in CommandLineOptions.cs. With this change, we can use the default help print from Commandline library. We do not need to write our own help text. Remove Operations.cs and Commandlinehelp.cs. I fold the functionality to program.cs and configgenerator.cs. Make options class immutable. https://github.com/commandlineparser/commandline/wiki/Immutable-Options-Type Make database type and host option explicit type from Enum instead of string. **Test** Manual testing. I don't change how the config is generated. Same variables are passed to config generator. * Update cosmosDB options (#29) * updated cosmosDB options * fix formatting * making code more modular * Update src/Models/CommandLineOptions.cs Co-authored-by: Aniruddh Munde <[email protected]> * updating test name * refactoring code * refactoring code Co-authored-by: Aniruddh Munde <[email protected]> * Move relationship related options to update. (#46) * Update README.md (#51) * Refactor config generator for Init and Add command (#41) * Add tests for initConfig and addEntity. Refactor code a bit to make it more readable. * Add comments. * More verbose with arg passing. * Format code. * Add more tests for AddEntity. * resolving comments * removing temp changes * small fix Co-authored-by: Abhishek Kumar <[email protected]> * fixed tests (#55) * Update README.md (#59) updating commands from --permission to --permissions * Simplify Update Entity logic (#52) * replace name to config * symplifying update logic * adding validations * added more tests * resolved comments * adding test description * updating file names for Add and Update EntityTests * adding comments and tests * adding class comments * adding class comments * adding more tests and comments * updated accessebiity of some methods * updating tests * adding TODO for policy support * updating few comments * updated function summary * minor change * fixed comments * refactoring changes * using csproj instead of dll files (#62) * Create CI pipeline (#56) * Create main.yml * Update main.yml * adding Datagateway.Config.csproj * fixing path * fixing tests * add tests validation in pipeline * adding formatting check * updating pipeline * removing temp changes * fix * fix * fix * fix * fix * fix * fix * fix * updating pipeline * updating pipeline files * updating pipeline files * updating pipeline * updating pipeline * updating pipeline * updating pipeline * fix pipeline * fix pipeline * fix pipeline * fix pipeline * fix pipeline * updating runner * updating runner * adding pipeline for ubuntu * fixing build job name * using matrix for running multiple os * updating .editorconfig * publishing code coverage * updating coverage directory * getting dir info * fixing pipeline * updating test script * fixing code coverage path * fixing code coverage path * fixing code coverage issue * fixing code coverage issue * fixing code coverage issue * adding deploy steps for the nuget package * adding deploy steps for the nuget package * add source path for nuget package * add source path for nuget package * fix source path for nuget package * fix test * updating format check * updating format check * updating format check * updating format check * updating format check * updating format check * updating format check * updating format check * testing submodules * fixing yaml * fixing yaml * fixing yaml * fixing yaml * updating .gitmodules * updating .gitmodules * updating .gitmodules * checking submodules * checking submodules * checking submodules * checking submodules * checking submodules * adding submodules * testing submodules * testing submodules * testing submodules * testing submodules * testing submodules * testing submodules * testing submodules * testing submodules * testing submodules * testing submodules * testing submodules * testing submodules * testing submodules * testing submodules * testing submodules * testing submodules * testing submodules * testing submodules * testing submodules * testing submodules * testing submodules * testing submodules * testing submodules * testing submodules * testing submodules * testing submodules * testing submodules * removing temp files * updating yaml * updating yaml * updating tests * updating tests * testing workflow_dispatch * testing workflow_dispatch * refactoring changes * testing coverage report to pr * testing coverage report to pr * testing coverage report to pr * testing coverage report to pr * testing coverage report to pr * testing coverage report to pr * testing coverage report to pr * fix coverage report to pr * fix coverage report to pr * fix coverage report to pr * testing different test reported * testing different test reported * testing coverage summary * fix pipeline * updating comments * fixing coverage report * removing temp files * removing submodules coverage * removing unused import * fixing pipeline * cleaning comments * Add support for custom graphql singular plural types (#65) * adding support for custom graphql singular plural types * fixed formatting * fixing pipeline * updating comments * removing redundant steps in pipeline * Adding test for Case Sensitive Entity Name (#64) * Adds test for entity creation * Adds change to create the options differently * Adding change to summary and comments * Removing jwt from the config json strings * Changing the default rest api endpoint * Builds config differently to avoid duplication * Fixing formatting issues * Preventing users from creation Permissions with Invalid Actions (#70) * adding tests * simplyfying code * added a new test * simplifying code * Changing true/false to be treated as a Boolean when used with --rest and --graphql options (#74) * Making true/false bool in --rest/graphql options * addressing review comments * checking in latest hawaii-engine commits * addressing review comments * adding a check for serialization of entity.graphql value * Add support for Policy in Permission Actions (#71) * added support for policy * fix formatting * added tests * fix build failure * fix formatting * adding comments * removed redundant code * updating tests * updating authentication provider * updating tests * updating editorconfig * updating submodules * updating submodules * fixing some dependencies * Add support for mappings (#76) * adding test * fix formatting * fix error * fix formatting * updating option description * updating comment * fixing build * Add support for cors origin (#77) * added support for cors-origin * fix formatting * fix test * fix formatting * Prevent user from adding relationship in cosmos db (#79) * preventing user from creating relationship in cosmosdb * updating test * Using IEnumerables for Options (#80) * adding changes for using ienumerables for options * removes redundant conversion * Adding checks for empty items * fixing console output messages * adding empty checks for ienumerable options in addition to null * checking in hawaii-engine sub-module * Checking in changes from engine * fixes formatting * adding test for update command with multiple mapping fields * fix formatting * removing extra spaces * incorporating PR review suggestions * checnking-in latest changes from hawaii engine * adding changes to remove empty mappings property in config json * minor correction in the test command * adding a test to validate the config json generated * minor correction in the test * New Release 0.0.4 (#84) * Fix serialization issue with special character (#87) * fixed serialization issue with special charachter * updating test * fix formatting * removing temp changes * fix test * updating test * updating test * updating test summary * fix pipeline * Renaming the option from --mapping.fields to --relationship.fields (#86) * renaming command option from mapping.fields to relationship.fields * changing sample commadns in README fiel * adding latest changes from hawaii engine * adding latest changes from engine * updating tests to use relationship.fields * merge cli into engine * updating csproj file * removing cli sln file * removing duplicate files * fix formatting * fix formatting * adding cli tests into build pipeline * fix format * updating folder name for cli-tests * updating csproj filename for hawaii-cli * updating csproj filename for hawaii-cli * fixing build * fix format * updating pipeline to create nuget package * publish nuget packe to artifact * fix build pipeline * testing * testing * testing * fixing path for exe * fixing path for exe * updating feed for nuget package * commenting nuget push * using dependency version from Directory.Build.Props * testing nuget push * fixing publish of nuget package * Update CLI to use Operation Enum. * Fix build break in tests. * Verify that operation is valid for action. * Remove unnessary using. * updating pipeline * updating nuget command in pipeline * updating nuget command in pipeline * updating nuget command in pipeline * updating nuget command in pipeline * testing nupkg creation * testing nupkg creation * testing nupkg creation * testing nupkg creation * testing nupkg creation * testing nupkg creation * testing nupkg creation * testing nupkg creation * testing nupkg creation * testing nupkg publishing * updating pipeline for nuget push * fixing pipeline * fixing pipeline * fixing pipeline * fixing pipeline * fixing pipeline * fixing pipeline * fixing pipeline * fixing pipeline * fix formatting * fix formatting Co-authored-by: Microsoft Open Source <[email protected]> Co-authored-by: Sajeetharan <[email protected]> Co-authored-by: Aniruddh Munde <[email protected]> Co-authored-by: Chris LaFreniere <[email protected]> Co-authored-by: Jarupat Jisarojito <[email protected]> Co-authored-by: Shyam Sundar J <[email protected]> Co-authored-by: Shyam Sundar J <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
We use HotChocolate in an uncommon way, this means documentation is
often of little help. So instead when we run into issues we need to read
the source code. Finding the right piece of code to read can be made a
lot easier if the debugger can show us what code is being executed. By
disabling 'Just My Code' the debugger will allow us to debug code from
the HotChocolate library that's being executed.