Skip to content

Merge release/dev16.8 to release/dev16.9#48593

Merged
24 commits merged intorelease/dev16.9from
merges/release/dev16.8-to-release/dev16.9
Oct 16, 2020
Merged

Merge release/dev16.8 to release/dev16.9#48593
24 commits merged intorelease/dev16.9from
merges/release/dev16.8-to-release/dev16.9

Conversation

@dotnet-bot
Copy link
Collaborator

This is an automatically generated pull request from release/dev16.8 into release/dev16.9.

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/release/dev16.8-to-release/dev16.9
git reset --hard upstream/release/dev16.9
git merge upstream/release/dev16.8
# Fix merge conflicts
git commit
git push upstream merges/release/dev16.8-to-release/dev16.9 --force

jasonmalinowski and others added 22 commits October 5, 2020 17:17
#47047 changed the logic for the
file name and path we generate, but we were no longer ensuring the
temporary directory existed prior to trying to create the file
within it.
The compiler added a block to prevent loading generators that are built
against the .NET Framework, since we know long term that we'll want them
to be runnable on .NET Core. The IDE helper that deals with converting
the error code to an IDE diagnostic needs to be updated to support this
new error, or otherwise Visual Studio will crash.

Fixes #47845
Update documentation for new minimum dependencies
…in SyntaxParser.Reset (#48393)

* Ensure that tokens are refetched to the position in the reset point in SyntaxParser.Reset

Fixes https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1179569.

* Fix spacing

* Change number for ERR_UnexpectedVarianceStaticMember
Get VS Color ThemeId without using dynamic
Avoid binding the world looking for target-typed new
…f-source-generated-file

Fix Go to Definition to a source-generated file
Handle disabled UndoManager during InlineRename
@dotnet-bot dotnet-bot requested review from a team as code owners October 14, 2020 18:02
@dotnet-bot dotnet-bot added Area-Infrastructure auto-merge Merge Conflicts There are merge conflicts with the base branch. labels Oct 14, 2020
@ghost
Copy link

ghost commented Oct 14, 2020

Hello @dotnet-bot!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

Do note that I've been instructed to only help merge pull requests of this repository that have been opened for at least 12 minutes. No worries though, I will be back when the time is right! 😉

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

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

@Cosifne
Copy link
Member

Cosifne commented Oct 14, 2020

@sharwell
This is the same as I mentioned here #48595

…ng-incompatible-source-generator

Don't crash if you reference a generator built against .NET Framework
@Cosifne Cosifne removed the Merge Conflicts There are merge conflicts with the base branch. label Oct 15, 2020
@Cosifne Cosifne assigned JoeRobich and unassigned sharwell Oct 15, 2020
@JoeRobich JoeRobich force-pushed the merges/release/dev16.8-to-release/dev16.9 branch from 69056d4 to 8746653 Compare October 15, 2020 20:25
@ghost ghost merged commit ac53515 into release/dev16.9 Oct 16, 2020
@ghost ghost deleted the merges/release/dev16.8-to-release/dev16.9 branch October 16, 2020 01:57
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants