Skip to content

Conversation

@dotnet-bot
Copy link
Contributor

This is an automatically generated pull request from master into dev15.6.

git fetch --all
git checkout merges/master-to-dev15.6-20171222-080050
git reset --hard upstream/dev15.6
git merge upstream/master
# Fix merge conflicts
git commit
git push merges/master-to-dev15.6-20171222-080050 --force

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

enricosada and others added 5 commits December 21, 2017 18:46
otherwise instead of expected `FSStrings.resources` will use `FSharp.Compiler.Service.netstandard.FSStrings.resources`
* Fix IndexOutOfRange in check for providing completion:

* Add test
* Filter incremental builder creation errors according to compiler args

* Fix CompilationErrorLogger ignores WarsAsError options

* Add test for WarnAsError

* Cleanup

* Add more tests; cover WarnAsError-, no warnings at all

* Refactor tests

* Add test
@KevinRansom KevinRansom merged commit 5f9e8df into dev15.6 Dec 22, 2017
@brettfo brettfo deleted the merges/master-to-dev15.6-20171222-080050 branch December 22, 2017 18:33
nosami pushed a commit to xamarin/visualfsharp that referenced this pull request Jan 26, 2022
* fix resource name (dotnet#4151)

otherwise instead of expected `FSStrings.resources` will use `FSharp.Compiler.Service.netstandard.FSStrings.resources`

* Use ConcurrentDictionary in ImportMap (dotnet#4148)

* Fix IndexOutOfRangeException in check for providing completion (dotnet#4138)

* Fix IndexOutOfRange in check for providing completion:

* Add test

* Remove repeating arguments processing in IncrementalBuilder creation (dotnet#4124)

* Report builder creation warnings according to compiler args (dotnet#4125)

* Filter incremental builder creation errors according to compiler args

* Fix CompilationErrorLogger ignores WarsAsError options

* Add test for WarnAsError

* Cleanup

* Add more tests; cover WarnAsError-, no warnings at all

* Refactor tests

* Add test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants