[GitHub] logging-log4net pull request #14: Add .NET Standard 2.0 build

2017-08-22 Thread chlowell
Github user chlowell closed the pull request at: https://github.com/apache/logging-log4net/pull/14 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featu

[GitHub] logging-log4net pull request #14: Add .NET Standard 2.0 build

2017-08-21 Thread chlowell
Github user chlowell commented on a diff in the pull request: https://github.com/apache/logging-log4net/pull/14#discussion_r134377119 --- Diff: tests/src/log4net.Tests.csproj --- @@ -19,245 +18,25 @@ under the License. --> -http://schemas.microsoft.com/developer

[GitHub] logging-log4net pull request #14: Add .NET Standard 2.0 build

2017-08-21 Thread chlowell
Github user chlowell commented on a diff in the pull request: https://github.com/apache/logging-log4net/pull/14#discussion_r134355177 --- Diff: tests/src/Context/LogicalThreadContextTest.cs --- @@ -17,7 +17,7 @@ // #endregion -#if NET_4_5 || NETSTANDARD1_3 +#

[GitHub] logging-log4net pull request #14: Add .NET Standard 2.0 build

2017-08-21 Thread chlowell
Github user chlowell commented on a diff in the pull request: https://github.com/apache/logging-log4net/pull/14#discussion_r134354215 --- Diff: src/CompatibilityExtensions.cs --- @@ -0,0 +1,53 @@ +#region Apache License +// +// Licensed to the Apache Software Foundati

[GitHub] logging-log4net pull request #14: Add .NET Standard 2.0 build

2017-08-18 Thread dpsenner
Github user dpsenner commented on a diff in the pull request: https://github.com/apache/logging-log4net/pull/14#discussion_r133930064 --- Diff: tests/src/log4net.Tests.csproj --- @@ -19,245 +18,25 @@ under the License. --> -http://schemas.microsoft.com/developer

[GitHub] logging-log4net pull request #14: Add .NET Standard 2.0 build

2017-08-18 Thread dpsenner
Github user dpsenner commented on a diff in the pull request: https://github.com/apache/logging-log4net/pull/14#discussion_r133929485 --- Diff: tests/src/Context/LogicalThreadContextTest.cs --- @@ -17,7 +17,7 @@ // #endregion -#if NET_4_5 || NETSTANDARD1_3 +#

[GitHub] logging-log4net pull request #14: Add .NET Standard 2.0 build

2017-08-18 Thread dpsenner
Github user dpsenner commented on a diff in the pull request: https://github.com/apache/logging-log4net/pull/14#discussion_r133929178 --- Diff: src/CompatibilityExtensions.cs --- @@ -0,0 +1,53 @@ +#region Apache License +// +// Licensed to the Apache Software Foundati

[GitHub] logging-log4net pull request #14: Add .NET Standard 2.0 build

2017-08-18 Thread dpsenner
Github user dpsenner commented on a diff in the pull request: https://github.com/apache/logging-log4net/pull/14#discussion_r133928541 --- Diff: Jenkinsfile --- @@ -61,21 +61,14 @@ pipeline { // run docker container

[GitHub] logging-log4net pull request #14: Add .NET Standard 2.0 build

2017-08-18 Thread dpsenner
Github user dpsenner commented on a diff in the pull request: https://github.com/apache/logging-log4net/pull/14#discussion_r133928219 --- Diff: src/Appender/AppenderSkeleton.cs --- @@ -321,7 +321,7 @@ public void DoAppend(LoggingEvent loggingEvent) {

[GitHub] logging-log4net pull request #14: Add .NET Standard 2.0 build

2017-08-17 Thread chlowell
Github user chlowell commented on a diff in the pull request: https://github.com/apache/logging-log4net/pull/14#discussion_r133848915 --- Diff: tests/src/log4net.Tests.csproj --- @@ -19,245 +18,25 @@ under the License. --> -http://schemas.microsoft.com/developer

[GitHub] logging-log4net pull request #14: Add .NET Standard 2.0 build

2017-08-17 Thread chlowell
Github user chlowell commented on a diff in the pull request: https://github.com/apache/logging-log4net/pull/14#discussion_r133846257 --- Diff: src/Appender/AppenderSkeleton.cs --- @@ -321,7 +321,7 @@ public void DoAppend(LoggingEvent loggingEvent) {

[GitHub] logging-log4net pull request #14: Add .NET Standard 2.0 build

2017-08-17 Thread chlowell
Github user chlowell commented on a diff in the pull request: https://github.com/apache/logging-log4net/pull/14#discussion_r133846136 --- Diff: src/CompatibilityExtensions.cs --- @@ -0,0 +1,53 @@ +#region Apache License +// +// Licensed to the Apache Software Foundati

[GitHub] logging-log4net pull request #14: Add .NET Standard 2.0 build

2017-08-17 Thread chlowell
Github user chlowell commented on a diff in the pull request: https://github.com/apache/logging-log4net/pull/14#discussion_r133845941 --- Diff: tests/src/Context/LogicalThreadContextTest.cs --- @@ -17,7 +17,7 @@ // #endregion -#if NET_4_5 || NETSTANDARD1_3 +#

[GitHub] logging-log4net pull request #14: Add .NET Standard 2.0 build

2017-08-17 Thread dpsenner
Github user dpsenner commented on a diff in the pull request: https://github.com/apache/logging-log4net/pull/14#discussion_r133704161 --- Diff: tests/src/log4net.Tests.csproj --- @@ -19,245 +18,25 @@ under the License. --> -http://schemas.microsoft.com/developer

[GitHub] logging-log4net pull request #14: Add .NET Standard 2.0 build

2017-08-17 Thread dpsenner
Github user dpsenner commented on a diff in the pull request: https://github.com/apache/logging-log4net/pull/14#discussion_r133700988 --- Diff: Jenkinsfile --- @@ -61,21 +61,14 @@ pipeline { // run docker container

[GitHub] logging-log4net pull request #14: Add .NET Standard 2.0 build

2017-08-17 Thread dpsenner
Github user dpsenner commented on a diff in the pull request: https://github.com/apache/logging-log4net/pull/14#discussion_r133703322 --- Diff: src/Util/PatternStringConverters/UserNamePatternConverter.cs --- @@ -69,6 +69,12 @@ override protected void Convert(TextWriter writer, obj

[GitHub] logging-log4net pull request #14: Add .NET Standard 2.0 build

2017-08-17 Thread dpsenner
Github user dpsenner commented on a diff in the pull request: https://github.com/apache/logging-log4net/pull/14#discussion_r133703847 --- Diff: tests/src/Context/LogicalThreadContextTest.cs --- @@ -17,7 +17,7 @@ // #endregion -#if NET_4_5 || NETSTANDARD1_3 +#

[GitHub] logging-log4net pull request #14: Add .NET Standard 2.0 build

2017-08-17 Thread dpsenner
Github user dpsenner commented on a diff in the pull request: https://github.com/apache/logging-log4net/pull/14#discussion_r133703032 --- Diff: src/CompatibilityExtensions.cs --- @@ -0,0 +1,53 @@ +#region Apache License +// +// Licensed to the Apache Software Foundati

[GitHub] logging-log4net pull request #14: Add .NET Standard 2.0 build

2017-08-17 Thread dpsenner
Github user dpsenner commented on a diff in the pull request: https://github.com/apache/logging-log4net/pull/14#discussion_r133701487 --- Diff: log4net.build --- @@ -1748,48 +1748,64 @@ limitations under the License. - - + +

[GitHub] logging-log4net pull request #14: Add .NET Standard 2.0 build

2017-08-17 Thread dpsenner
Github user dpsenner commented on a diff in the pull request: https://github.com/apache/logging-log4net/pull/14#discussion_r133702415 --- Diff: src/Appender/AppenderSkeleton.cs --- @@ -321,7 +321,7 @@ public void DoAppend(LoggingEvent loggingEvent) {

[GitHub] logging-log4net pull request #14: Add .NET Standard 2.0 build

2017-08-16 Thread chlowell
GitHub user chlowell opened a pull request: https://github.com/apache/logging-log4net/pull/14 Add .NET Standard 2.0 build This adds a netstandard2.0 build consumable on .NET Core 2.0, .NET Framework 4.6.1+, Xamarin platforms, and the next version of UWP ([this doc](https://docs.mic