This repository was archived by the owner on Jan 12, 2024. It is now read-only.
Merge Master into Feature Branch #591
Merged
Conversation
This file contains hidden or 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
…icts (#566) This change removes the dependency on NuGet.ProjectModel from the LanguageServer project and instead relies on the installed version of the binary. This change intends to avoid the dependency between the .Net Core SDK version and NuGet in MSBuildLocator. See issue #86 for additional reference. (microsoft/MSBuildLocator#86)
…` twice (#570) * Get qsc command exit code. * Delete bson file when qsc command fails. * Fail build when qsc fails. * Update src/QuantumSdk/Sdk/Sdk.targets Co-authored-by: Andres Paz <anpaz@microsoft.com> Co-authored-by: Andres Paz <anpaz@microsoft.com>
…#577) Through a recent component governance audit we're extending the list of third party libraries used by components in this repository.
* Auto-create .csproj file if none exists * Move temporary project to template file * Refactor and add unit test * Remove some duplication * Fixed version of Microsoft.Quantum.Sdk for test * Stable filename for temporary .csproj * Format hash code as hex * Ensure FileSystemWatcher for temporary project folder * Only watch .csproj in temporary folder * Use lower-case folder name Co-authored-by: Ricardo Espinoza <43434922+ricardo-espinoza@users.noreply.github.com>
* Split into two project manager queries * Remove Wait() call * Revert "Remove Wait() call" This reverts commit f149e9e. * Add QsCompilerError.Verify call * Apply suggestions from code review Co-authored-by: bettinaheim <34236215+bettinaheim@users.noreply.github.com> Co-authored-by: bettinaheim <34236215+bettinaheim@users.noreply.github.com>
* Add ExposeReferencesViaTestNames to assemblyProperties Allows ExposeReferencesViaTestNames to be passed to rewrite steps * Add ExposeReferencesViaTestNames to AssemblyConstants
Include nested self-resolutions for type parameters as constrictive resolutions.
* Unify position and range types in parser and syntax tree (#523) * Add new Position and Range types * Require line and column to be non-negative * Require start <= end * Replace private record with DU case * Add documentation * Fix sln file * Remove custom equality/comparison * Add backwards-compatible JSON serializers * Disable Example test target for now * Fix RangePosition serialization * Generate valid ranges in SymbolManagementTests * Skip execution tests for now * Fix null offset in QsDeclarationAttribute * Fix position + range * Remove range + position * Write private constructor on one line * Rename Range.Combine to Range.Span * Add more documentation * Use 1-based range positions in serialization * Re-enable C# generation tests * Disable execution tests again * Revert "Disable execution tests again" This reverts commit ec0273f. * Update package references * Subtract 1 from all range positions in SerializationTests * Replace TryGetFileId with GetFileId in SerializationTests * Rename id to ident * Replace some LSP positions with Q# positions * Replace LSP positions in DataStructures.cs * Replace LSP positions in ContextBuilder.cs * Replace LSP positions in TextProcessor.cs * Replace LSP positions in TypeChecking.cs * Remove type parameters from SymbolTracker and ScopeContext * Replace LSP positions in CodeActions.cs * Replace LSP positions in CodeCompletion.cs * Replace LSP positions in EditorCommands.cs * Replace LSP positions in Diagnostics.cs * Remove unused functions * Remove more unnecessary functions * Remove IsSmallerThan and IsSmallerThanOrEqualTo * Add Range.Contains * Update docs and variable names for + and - * Convert some LSP ranges to Q# ranges * Rename IsValidPosition/Range with file to ContainsPosition/Range * Remove IsValidPosition and IsValidRange * Replace more ranges * Avoid unnecessary conversion * Replace GetRange() with Range * Use properties for positions in TreeNode * Replace HeaderEntry.GetPosition() with property * Fix build errors after merge * Add position in CheckDefinedTypesForCycles and ResolveAll * Add positions in verifyResultConditionalBlocks * Add positions in NewConjugation * Add positions in AllPathsReturnValueOrFail * Use lambda in CheckDefinedTypesForCycles * Update range type * Use original OnRangeInformation names * Use ContainsEnd * Specify end point is excluded in doc comment * Preserve comments * Rename TranslateLines to WithLineNumOffset
ScottCarda-MS
requested review from
bamarsha,
bettinaheim,
cesarzc and
swernli
bamarsha
approved these changes
Aug 26, 2020
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Merges master into the feature branch.