Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Merge main to main-vs-deps #54230

Merged
46 commits merged into from
Jun 19, 2021
Merged

Merge main to main-vs-deps #54230

46 commits merged into from
Jun 19, 2021

Conversation

dotnet-bot
Copy link
Collaborator

This is an automatically generated pull request from main into main-vs-deps.

Once all conflicts are resolved and all the tests pass, you are free to merge the pull request. 🐯

Troubleshooting conflicts

Identify authors of changes which introduced merge conflicts

Scroll to the bottom, then for each file containing conflicts copy its path into the following searches:

Usually the most recent change to a file between the two branches is considered to have introduced the conflicts, but sometimes it will be necessary to look for the conflicting lines and check the blame in each branch. Generally the author whose change introduced the conflicts should pull down this PR, fix the conflicts locally, then push up a commit resolving the conflicts.

Resolve merge conflicts using your local repo

Sometimes merge conflicts may be present on GitHub but merging locally will work without conflicts. This is due to differences between the merge algorithm used in local git versus the one used by GitHub.

git fetch --all
git checkout merges/main-to-main-vs-deps
git reset --hard upstream/main-vs-deps
git merge upstream/main
# Fix merge conflicts
git commit
git push upstream merges/main-to-main-vs-deps --force

dibarbet and others added 30 commits May 27, 2021 23:52
…ike a resultId we can pass between the client and server
…ed builds inside Visual Studio

Work towards [AB#1337109](https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1337109)

For builds which are indirectly triggered inside Visual Studio from commands such as 'Run Tests' or 'Start Debugging', we implicitly skip analyzers to speed up these builds.

Such builds will set a special property `IsIndirectlyTriggeredBuildInsideVisualStudio` to `true`, which guards the implicit skip analyzers logic. We display a special message to inform the users about us implicitly skipping analyzers to speed up the build.

To enable MSBuild's incremental build logic and project system's fast-upto-date check logic to work correctly, we create/touch a special semaphore file to indicate the time stamp for last build with skipAnalyzers flag. This semaphore file is passed as a custom additional file input to builds without skipAnalyzers flag to ensure correct incremental builds. Additionally, we pass this file as an 'UpToDateCheckInput' item for project system's fast-upto-date check.
sharwell and others added 14 commits June 17, 2021 11:59
Ideally ImmutableSegmentedHashSet<string> would be used, but this type
is not currently available. As an interim solution,
ImmutableSegmentedDictionary<string, VoidResult> is used as a set.

See https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1343268
Use segmented collection for faster declaration cache
Add functionality to enable skipping analyzers for implicitly trigger…
* Allow snippet format in XAML compeltion

* change InsertSnippet to IsSnippet

* fix formatting
Handle tokens with Kind == None in argument completion
Optimize calculation of remote supported languages
Initialize MEF services before switching to the main thread
Use isIncomplete flag to cap the LSP completion list size for faster serialization
… 20210618.1 (#54219)

[main] Update dependencies from dotnet/source-build
@dotnet-bot dotnet-bot requested review from a team as code owners June 18, 2021 22:59
@dotnet-bot dotnet-bot added Area-Infrastructure auto-merge Merge Conflicts There are merge conflicts with the base branch. labels Jun 18, 2021
@dotnet-bot
Copy link
Collaborator Author

⚠ This PR has merge conflicts. @RikkiGibson

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Auto-approval

@genlu
Copy link
Member

genlu commented Jun 18, 2021

@dibarbet I resolved the merge conflicts by using 17.0 version of Microsoft.VisualStudio.LanguageServer.Protocol that was in main-vs-deps before. If it doesn't have the change required, please make sure to update it to a newer 17.0 version in main-vs-deps. Thanks!
36f5fb6

@ghost ghost merged commit 934166c into main-vs-deps Jun 19, 2021
@ghost ghost deleted the merges/main-to-main-vs-deps branch June 19, 2021 00:03
@ghost ghost added this to the Next milestone Jun 19, 2021
@RikkiGibson RikkiGibson modified the milestones: Next, 17.0.P2 Jun 29, 2021
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Infrastructure auto-merge Merge Conflicts There are merge conflicts with the base branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants