Skip to content

Conversation

@dotnet-bot
Copy link
Contributor

This is an automatically generated pull request from main into release/dev17.5.

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

nojaf and others added 5 commits October 7, 2022 20:16
…pe.Tuple (#13850)

* Tests added for range of SynType.Tuple
* Adding parser support for N-d arrays up to rank 32
* Update to fantomas 5.0.3. 
* Expend ignore patterns.
* Pattern discard not allowed for union case that takes no data

* Add test for grouped patterns discard

* Remove unnecesary use of wildcare across the compiler

* Format code

* Remove unnecesary wildcard on tests

* Update the warning logic

* Add warning behing preview flag

* More testing

* Add single-case unions when using them as a deconstruct syntax in functions

* Update FSComp.txt

Co-authored-by: Tomas Grosup <[email protected]>
@dotnet-bot dotnet-bot merged commit 5438a7c into release/dev17.5 Oct 11, 2022
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.

6 participants