Skip to content

Merge release/dev17.3 to release/dev17.4#63282

Merged
JoeRobich merged 6 commits intorelease/dev17.4from
merges/release/dev17.3-to-release/dev17.4
Aug 12, 2022
Merged

Merge release/dev17.3 to release/dev17.4#63282
JoeRobich merged 6 commits intorelease/dev17.4from
merges/release/dev17.3-to-release/dev17.4

Conversation

@dotnet-bot
Copy link
Copy Markdown
Collaborator

This is an automatically generated pull request from release/dev17.3 into release/dev17.4.

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 -t upstream/merges/release/dev17.3-to-release/dev17.4
git reset --hard upstream/release/dev17.4
git merge upstream/release/dev17.3
# Fix merge conflicts
git commit
git push upstream merges/release/dev17.3-to-release/dev17.4 --force

@dotnet-bot dotnet-bot requested review from a team as code owners August 9, 2022 21:01
@dotnet-bot dotnet-bot added Area-Infrastructure auto-merge Merge Conflicts There are merge conflicts with the base branch. labels Aug 9, 2022
@dotnet-bot
Copy link
Copy Markdown
Collaborator Author

⚠ This PR has merge conflicts. @dibarbet

@dotnet-bot dotnet-bot enabled auto-merge August 9, 2022 21:01
Copy link
Copy Markdown

@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

@dibarbet
Copy link
Copy Markdown
Member

@333fred could you take a quick look at these conflicts?

dibarbet and others added 5 commits August 9, 2022 18:07
Backport some integration test skips to 17.3
Revert "Merge pull request #61568 from CyrusNajmabadi/localFuncindexing"
…elease/dev17.3-to-release/dev17.4

* upstream/release/dev17.3:
  Revert "Merge pull request #61568 from CyrusNajmabadi/localFuncindexing"
  Backport some integration test skips to 17.3
  Mark shipped APIs (#63244)
@JoeRobich JoeRobich disabled auto-merge August 12, 2022 22:53
@JoeRobich JoeRobich merged commit d69f080 into release/dev17.4 Aug 12, 2022
@333fred 333fred deleted the merges/release/dev17.3-to-release/dev17.4 branch August 12, 2022 23:21
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.

4 participants