Skip to content

Commit

Permalink
Merge pull request #663 from nventive/dev/mafi/stable-z-vs15.9-compat…
Browse files Browse the repository at this point in the history
…ibility

Add missing analyzer for VS15.9 compatibility
  • Loading branch information
MatFillion authored Mar 8, 2019
2 parents ec61215 + 7acb7d9 commit 2ee6e1f
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion gitversion.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
assembly-versioning-scheme: MajorMinorPatch
mode: ContinuousDeployment
next-version: 1.43.0
next-version: 1.43.1
continuous-delivery-fallback-tag: ""
branches:
master:
Expand Down
2 changes: 1 addition & 1 deletion src/SourceGenerators/System.Xaml/System.Xaml/XamlType.cs
Original file line number Diff line number Diff line change
Expand Up @@ -467,7 +467,7 @@ IEnumerable<XamlMember> DoLookupAllMembers ()
var bf = BindingFlags.Public | BindingFlags.NonPublic | BindingFlags.Instance;

foreach (var pi in UnderlyingType.GetProperties (bf)) {
if (pi.Name.Contains ('.')) // exclude explicit interface implementations.
if (pi.Name.Contains (".")) // exclude explicit interface implementations.
continue;
if (pi.CanRead && (pi.CanWrite || IsCollectionType (pi.PropertyType) || typeof (IXmlSerializable).IsAssignableFrom (pi.PropertyType)) && pi.GetIndexParameters ().Length == 0)
yield return new XamlMember (pi, SchemaContext);
Expand Down
1 change: 1 addition & 0 deletions src/Uno.MonoAnalyzers/MonoNotSupportedAPIAnalyzer.cs
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,7 @@ public Container(Compilation compilation, MonoNotSupportedAPIAnalyzer owner)
"IndexOfAny",
"Join",
"StartsWith",
"Contains",
},
Validation = new Func<IMethodSymbol, bool>(
m => m.Parameters.FirstOrDefault()?.Type == _charSymbol
Expand Down

0 comments on commit 2ee6e1f

Please sign in to comment.