Skip to content
This repository was archived by the owner on Jan 12, 2024. It is now read-only.

Conversation

@ricardo-espinoza
Copy link
Contributor

No description provided.

bamarsha and others added 11 commits June 22, 2020 13:49
…449)

* Prevent result equality with QPRGen0

* Move more fields from SymbolTracker to ResolutionContext

* Undo some changes to TypeChecking.cs

* Move ResolutionContext up a bit

* Better error message

* Add basic tests

* Add more tests (but not working right now...)

* Fix tests

* Add tests for not equals

* Use static member for Create

* Move ResolutionContext to SymbolTracker.fs

* Update doc comment to ArgumentException

Co-authored-by: bettinaheim <34236215+bettinaheim@users.noreply.github.com>

* Replace failwith with ArgumentException

* Mention SymbolTracker versioning in ResolutionContext

* Restore supportsEqualityComparison and use VerifyIsOneOf

* Undo changes to VerificationTools.fs

* Rename ResolutionContext to ScopeContext

* Include execution target in error message

* Make ScopeContext.ExecutionTarget non-nullable

* Update error message

Co-authored-by: bettinaheim <34236215+bettinaheim@users.noreply.github.com>
Reordered the steps of the classically controlled rewrite to have the if-statement restructuring done first.
* Use Uri.LocalPath instead of Uri.AbsolutePath

* Fix attribute reader test

* Update packages to fix tests

* Add GetFileId and deprecate TryGetFileId

* Re-add null URI check

Co-authored-by: bettinaheim <34236215+bettinaheim@users.noreply.github.com>
Added NoOp to precondition dependency list for classical control.
Removed NoOp to precondition dependency list for classical control.
@ricardo-espinoza ricardo-espinoza merged commit 1d1988e into features/yeoman-templates Jun 29, 2020
@ricardo-espinoza ricardo-espinoza deleted the ricardoe/merge-master/2020-06-29-14-40 branch July 14, 2020 06:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants