From 862c939a3a2da3f0d218d0fc2ef9eacaf39a6b30 Mon Sep 17 00:00:00 2001 From: Stefan Nikolei Date: Sat, 23 Jan 2021 14:15:32 +0100 Subject: [PATCH 01/10] Add net5.0 target * add it in ImageSharp.csproj * update Buildscript --- .github/workflows/build-and-test.yml | 12 ++++++++++++ src/ImageSharp/ImageSharp.csproj | 2 +- 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/.github/workflows/build-and-test.yml b/.github/workflows/build-and-test.yml index 9e760c4b23..778e27e51f 100644 --- a/.github/workflows/build-and-test.yml +++ b/.github/workflows/build-and-test.yml @@ -14,6 +14,18 @@ jobs: strategy: matrix: options: + - os: ubuntu-latest + framework: net5.0 + runtime: -x64 + codecov: true + - os: macos-latest + framework: net5.0 + runtime: -x64 + codecov: false + - os: windows-latest + framework: net5.0 + runtime: -x64 + codecov: false - os: ubuntu-latest framework: netcoreapp3.1 runtime: -x64 diff --git a/src/ImageSharp/ImageSharp.csproj b/src/ImageSharp/ImageSharp.csproj index a90aaf715a..ea32661ddb 100644 --- a/src/ImageSharp/ImageSharp.csproj +++ b/src/ImageSharp/ImageSharp.csproj @@ -12,7 +12,7 @@ $(RepositoryUrl) Image Resize Crop Gif Jpg Jpeg Bitmap Png Tga NetCore A new, fully featured, fully managed, cross-platform, 2D graphics API for .NET - netcoreapp3.1;netcoreapp2.1;netstandard2.1;netstandard2.0;netstandard1.3;net472 + net5.0;netcoreapp3.1;netcoreapp2.1;netstandard2.1;netstandard2.0;netstandard1.3;net472 From 72037f46f80781fb99e3906da61f3f7196b1a97e Mon Sep 17 00:00:00 2001 From: Stefan Nikolei Date: Sat, 23 Jan 2021 14:19:16 +0100 Subject: [PATCH 02/10] add .net5.0 targetframework in tests --- tests/ImageSharp.Benchmarks/ImageSharp.Benchmarks.csproj | 2 +- .../ImageSharp.Tests.ProfilingSandbox.csproj | 2 +- tests/ImageSharp.Tests/ImageSharp.Tests.csproj | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/ImageSharp.Benchmarks/ImageSharp.Benchmarks.csproj b/tests/ImageSharp.Benchmarks/ImageSharp.Benchmarks.csproj index 2a209683ba..d89cf79fc9 100644 --- a/tests/ImageSharp.Benchmarks/ImageSharp.Benchmarks.csproj +++ b/tests/ImageSharp.Benchmarks/ImageSharp.Benchmarks.csproj @@ -5,7 +5,7 @@ ImageSharp.Benchmarks Exe SixLabors.ImageSharp.Benchmarks - netcoreapp3.1;netcoreapp2.1;net472 + net5.0;netcoreapp3.1;netcoreapp2.1;net472 false false diff --git a/tests/ImageSharp.Tests.ProfilingSandbox/ImageSharp.Tests.ProfilingSandbox.csproj b/tests/ImageSharp.Tests.ProfilingSandbox/ImageSharp.Tests.ProfilingSandbox.csproj index b71b5ebf2f..220780c58e 100644 --- a/tests/ImageSharp.Tests.ProfilingSandbox/ImageSharp.Tests.ProfilingSandbox.csproj +++ b/tests/ImageSharp.Tests.ProfilingSandbox/ImageSharp.Tests.ProfilingSandbox.csproj @@ -8,7 +8,7 @@ false SixLabors.ImageSharp.Tests.ProfilingSandbox win7-x64 - netcoreapp3.1;netcoreapp2.1;net472 + net5.0;netcoreapp3.1;netcoreapp2.1;net472 SixLabors.ImageSharp.Tests.ProfilingSandbox.Program false diff --git a/tests/ImageSharp.Tests/ImageSharp.Tests.csproj b/tests/ImageSharp.Tests/ImageSharp.Tests.csproj index 5426144401..ebbe2cbdfa 100644 --- a/tests/ImageSharp.Tests/ImageSharp.Tests.csproj +++ b/tests/ImageSharp.Tests/ImageSharp.Tests.csproj @@ -2,7 +2,7 @@ - netcoreapp3.1;netcoreapp2.1;net472 + net5.0;netcoreapp3.1;netcoreapp2.1;net472 True SixLabors.ImageSharp.Tests AnyCPU;x64;x86 From 58491c9c12fa2ee44c2fb976b2465ea6936a4d1f Mon Sep 17 00:00:00 2001 From: Stefan Nikolei Date: Sat, 23 Jan 2021 16:38:18 +0100 Subject: [PATCH 03/10] remove .net50 target framework --- src/ImageSharp/ImageSharp.csproj | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/ImageSharp/ImageSharp.csproj b/src/ImageSharp/ImageSharp.csproj index ea32661ddb..a90aaf715a 100644 --- a/src/ImageSharp/ImageSharp.csproj +++ b/src/ImageSharp/ImageSharp.csproj @@ -12,7 +12,7 @@ $(RepositoryUrl) Image Resize Crop Gif Jpg Jpeg Bitmap Png Tga NetCore A new, fully featured, fully managed, cross-platform, 2D graphics API for .NET - net5.0;netcoreapp3.1;netcoreapp2.1;netstandard2.1;netstandard2.0;netstandard1.3;net472 + netcoreapp3.1;netcoreapp2.1;netstandard2.1;netstandard2.0;netstandard1.3;net472 From 3fad302539a178a72703fb425f7d845dd92c95a2 Mon Sep 17 00:00:00 2001 From: Stefan Nikolei Date: Sat, 23 Jan 2021 16:42:54 +0100 Subject: [PATCH 04/10] codecov only on one target --- .github/workflows/build-and-test.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/build-and-test.yml b/.github/workflows/build-and-test.yml index 778e27e51f..4da7813a8a 100644 --- a/.github/workflows/build-and-test.yml +++ b/.github/workflows/build-and-test.yml @@ -17,7 +17,7 @@ jobs: - os: ubuntu-latest framework: net5.0 runtime: -x64 - codecov: true + codecov: false - os: macos-latest framework: net5.0 runtime: -x64 From 37709f941e4371cb3d99dd16f5739c9e1b9d7d55 Mon Sep 17 00:00:00 2001 From: Stefan Nikolei Date: Mon, 25 Jan 2021 15:34:41 +0100 Subject: [PATCH 05/10] test and build framework can be different --- .github/workflows/build-and-test.yml | 31 ++++++++++++++++++---------- 1 file changed, 20 insertions(+), 11 deletions(-) diff --git a/.github/workflows/build-and-test.yml b/.github/workflows/build-and-test.yml index 4da7813a8a..579a547ea6 100644 --- a/.github/workflows/build-and-test.yml +++ b/.github/workflows/build-and-test.yml @@ -15,39 +15,48 @@ jobs: matrix: options: - os: ubuntu-latest - framework: net5.0 + framework-build: netcoreapp3.1 + framework-test: net5.0 runtime: -x64 codecov: false - os: macos-latest - framework: net5.0 + framework-build: netcoreapp3.1 + framework-test: net5.0 runtime: -x64 codecov: false - os: windows-latest - framework: net5.0 + framework-build: netcoreapp3.1 + framework-test: net5.0 runtime: -x64 codecov: false - os: ubuntu-latest - framework: netcoreapp3.1 + framework-build: netcoreapp3.1 + framework-test: netcoreapp3.1 runtime: -x64 codecov: true - os: macos-latest - framework: netcoreapp3.1 + framework-build: netcoreapp3.1 + framework-test: netcoreapp3.1 runtime: -x64 codecov: false - os: windows-latest - framework: netcoreapp3.1 + framework-build: netcoreapp3.1 + framework-test: netcoreapp3.1 runtime: -x64 codecov: false - os: windows-latest - framework: netcoreapp2.1 + framework-build: netcoreapp2.1 + framework-test: netcoreapp2.1 runtime: -x64 codecov: false - os: windows-latest - framework: net472 + framework-build: net472 + framework-test: net472 runtime: -x64 codecov: false - os: windows-latest - framework: net472 + framework-build: net472 + framework-test: net472 runtime: -x86 codecov: false @@ -78,11 +87,11 @@ jobs: - name: Build shell: pwsh - run: ./ci-build.ps1 "${{matrix.options.framework}}" + run: ./ci-build.ps1 "${{matrix.options.framework-build}}" - name: Test shell: pwsh - run: ./ci-test.ps1 "${{matrix.options.os}}" "${{matrix.options.framework}}" "${{matrix.options.runtime}}" "${{matrix.options.codecov}}" + run: ./ci-test.ps1 "${{matrix.options.os}}" "${{matrix.options.framework-test}}" "${{matrix.options.runtime}}" "${{matrix.options.codecov}}" env: CI: True XUNIT_PATH: .\tests\ImageSharp.Tests # Required for xunit From e5f640182c692cf7030669c3fe65fcdad5782a1b Mon Sep 17 00:00:00 2001 From: Stefan Nikolei Date: Wed, 27 Jan 2021 21:52:01 +0100 Subject: [PATCH 06/10] use CI environment variable to also build on .net5.0 --- .github/workflows/build-and-test.yml | 31 ++++++++++------------------ src/ImageSharp/ImageSharp.csproj | 7 +++++++ 2 files changed, 18 insertions(+), 20 deletions(-) diff --git a/.github/workflows/build-and-test.yml b/.github/workflows/build-and-test.yml index 579a547ea6..4da7813a8a 100644 --- a/.github/workflows/build-and-test.yml +++ b/.github/workflows/build-and-test.yml @@ -15,48 +15,39 @@ jobs: matrix: options: - os: ubuntu-latest - framework-build: netcoreapp3.1 - framework-test: net5.0 + framework: net5.0 runtime: -x64 codecov: false - os: macos-latest - framework-build: netcoreapp3.1 - framework-test: net5.0 + framework: net5.0 runtime: -x64 codecov: false - os: windows-latest - framework-build: netcoreapp3.1 - framework-test: net5.0 + framework: net5.0 runtime: -x64 codecov: false - os: ubuntu-latest - framework-build: netcoreapp3.1 - framework-test: netcoreapp3.1 + framework: netcoreapp3.1 runtime: -x64 codecov: true - os: macos-latest - framework-build: netcoreapp3.1 - framework-test: netcoreapp3.1 + framework: netcoreapp3.1 runtime: -x64 codecov: false - os: windows-latest - framework-build: netcoreapp3.1 - framework-test: netcoreapp3.1 + framework: netcoreapp3.1 runtime: -x64 codecov: false - os: windows-latest - framework-build: netcoreapp2.1 - framework-test: netcoreapp2.1 + framework: netcoreapp2.1 runtime: -x64 codecov: false - os: windows-latest - framework-build: net472 - framework-test: net472 + framework: net472 runtime: -x64 codecov: false - os: windows-latest - framework-build: net472 - framework-test: net472 + framework: net472 runtime: -x86 codecov: false @@ -87,11 +78,11 @@ jobs: - name: Build shell: pwsh - run: ./ci-build.ps1 "${{matrix.options.framework-build}}" + run: ./ci-build.ps1 "${{matrix.options.framework}}" - name: Test shell: pwsh - run: ./ci-test.ps1 "${{matrix.options.os}}" "${{matrix.options.framework-test}}" "${{matrix.options.runtime}}" "${{matrix.options.codecov}}" + run: ./ci-test.ps1 "${{matrix.options.os}}" "${{matrix.options.framework}}" "${{matrix.options.runtime}}" "${{matrix.options.codecov}}" env: CI: True XUNIT_PATH: .\tests\ImageSharp.Tests # Required for xunit diff --git a/src/ImageSharp/ImageSharp.csproj b/src/ImageSharp/ImageSharp.csproj index a90aaf715a..e0da2abf0d 100644 --- a/src/ImageSharp/ImageSharp.csproj +++ b/src/ImageSharp/ImageSharp.csproj @@ -12,9 +12,16 @@ $(RepositoryUrl) Image Resize Crop Gif Jpg Jpeg Bitmap Png Tga NetCore A new, fully featured, fully managed, cross-platform, 2D graphics API for .NET + + + netcoreapp3.1;netcoreapp2.1;netstandard2.1;netstandard2.0;netstandard1.3;net472 + + net5.0;netcoreapp3.1;netcoreapp2.1;netstandard2.1;netstandard2.0;netstandard1.3;net472 + + From fb508475ebe20057391c88006b02489470050fa7 Mon Sep 17 00:00:00 2001 From: Jeff Skaistis Date: Sat, 30 Jan 2021 11:49:47 -0600 Subject: [PATCH 07/10] Change size guards for WrapMemory to allow Memory/IMemoryOwner blocks larger than the required image size. --- src/ImageSharp/Image.WrapMemory.cs | 30 ++++++------ .../Image/ImageTests.WrapMemory.cs | 48 ++++++++++++++++--- 2 files changed, 57 insertions(+), 21 deletions(-) diff --git a/src/ImageSharp/Image.WrapMemory.cs b/src/ImageSharp/Image.WrapMemory.cs index 383f643969..b0efdb60db 100644 --- a/src/ImageSharp/Image.WrapMemory.cs +++ b/src/ImageSharp/Image.WrapMemory.cs @@ -17,7 +17,7 @@ namespace SixLabors.ImageSharp { /// /// - /// Wraps an existing contiguous memory area of 'width' x 'height' pixels allowing viewing/manipulation as + /// Wraps an existing contiguous memory area of at least 'width' x 'height' pixels allowing viewing/manipulation as /// an instance. /// /// @@ -52,7 +52,7 @@ namespace SixLabors.ImageSharp { Guard.NotNull(configuration, nameof(configuration)); Guard.NotNull(metadata, nameof(metadata)); - Guard.IsTrue(pixelMemory.Length == width * height, nameof(pixelMemory), "The length of the input memory doesn't match the specified image size"); + Guard.IsTrue(pixelMemory.Length >= width * height, nameof(pixelMemory), "The length of the input memory is less than the specified image size"); var memorySource = MemoryGroup.Wrap(pixelMemory); return new Image(configuration, memorySource, width, height, metadata); @@ -60,7 +60,7 @@ namespace SixLabors.ImageSharp /// /// - /// Wraps an existing contiguous memory area of 'width' x 'height' pixels allowing viewing/manipulation as + /// Wraps an existing contiguous memory area of at least 'width' x 'height' pixels allowing viewing/manipulation as /// an instance. /// /// @@ -93,7 +93,7 @@ namespace SixLabors.ImageSharp /// /// - /// Wraps an existing contiguous memory area of 'width' x 'height' pixels allowing viewing/manipulation as + /// Wraps an existing contiguous memory area of at least 'width' x 'height' pixels allowing viewing/manipulation as /// an instance. /// /// @@ -122,7 +122,7 @@ namespace SixLabors.ImageSharp => WrapMemory(Configuration.Default, pixelMemory, width, height); /// - /// Wraps an existing contiguous memory area of 'width' x 'height' pixels, + /// Wraps an existing contiguous memory area of at least 'width' x 'height' pixels, /// allowing to view/manipulate it as an instance. /// The ownership of the is being transferred to the new instance, /// meaning that the caller is not allowed to dispose . @@ -147,14 +147,14 @@ namespace SixLabors.ImageSharp { Guard.NotNull(configuration, nameof(configuration)); Guard.NotNull(metadata, nameof(metadata)); - Guard.IsTrue(pixelMemoryOwner.Memory.Length == width * height, nameof(pixelMemoryOwner), "The length of the input memory doesn't match the specified image size"); + Guard.IsTrue(pixelMemoryOwner.Memory.Length >= width * height, nameof(pixelMemoryOwner), "The length of the input memory is less than the specified image size"); var memorySource = MemoryGroup.Wrap(pixelMemoryOwner); return new Image(configuration, memorySource, width, height, metadata); } /// - /// Wraps an existing contiguous memory area of 'width' x 'height' pixels, + /// Wraps an existing contiguous memory area of at least 'width' x 'height' pixels, /// allowing to view/manipulate it as an instance. /// The ownership of the is being transferred to the new instance, /// meaning that the caller is not allowed to dispose . @@ -176,7 +176,7 @@ namespace SixLabors.ImageSharp => WrapMemory(configuration, pixelMemoryOwner, width, height, new ImageMetadata()); /// - /// Wraps an existing contiguous memory area of 'width' x 'height' pixels, + /// Wraps an existing contiguous memory area of at least 'width' x 'height' pixels, /// allowing to view/manipulate it as an instance. /// The ownership of the is being transferred to the new instance, /// meaning that the caller is not allowed to dispose . @@ -196,7 +196,7 @@ namespace SixLabors.ImageSharp /// /// - /// Wraps an existing contiguous memory area of 'width' x 'height' pixels allowing viewing/manipulation as + /// Wraps an existing contiguous memory area of at least 'width' x 'height' pixels allowing viewing/manipulation as /// an instance. /// /// @@ -234,7 +234,7 @@ namespace SixLabors.ImageSharp var memoryManager = new ByteMemoryManager(byteMemory); - Guard.IsTrue(memoryManager.Memory.Length == width * height, nameof(byteMemory), "The length of the input memory doesn't match the specified image size"); + Guard.IsTrue(memoryManager.Memory.Length >= width * height, nameof(byteMemory), "The length of the input memory is less than the specified image size"); var memorySource = MemoryGroup.Wrap(memoryManager.Memory); return new Image(configuration, memorySource, width, height, metadata); @@ -242,7 +242,7 @@ namespace SixLabors.ImageSharp /// /// - /// Wraps an existing contiguous memory area of 'width' x 'height' pixels allowing viewing/manipulation as + /// Wraps an existing contiguous memory area of at least 'width' x 'height' pixels allowing viewing/manipulation as /// an instance. /// /// @@ -275,7 +275,7 @@ namespace SixLabors.ImageSharp /// /// - /// Wraps an existing contiguous memory area of 'width' x 'height' pixels allowing viewing/manipulation as + /// Wraps an existing contiguous memory area of at least 'width' x 'height' pixels allowing viewing/manipulation as /// an instance. /// /// @@ -305,7 +305,7 @@ namespace SixLabors.ImageSharp /// /// - /// Wraps an existing contiguous memory area of 'width' x 'height' pixels allowing viewing/manipulation as + /// Wraps an existing contiguous memory area of at least 'width' x 'height' pixels allowing viewing/manipulation as /// an instance. /// /// @@ -355,7 +355,7 @@ namespace SixLabors.ImageSharp /// /// - /// Wraps an existing contiguous memory area of 'width' x 'height' pixels allowing viewing/manipulation as + /// Wraps an existing contiguous memory area of at least 'width' x 'height' pixels allowing viewing/manipulation as /// an instance. /// /// @@ -393,7 +393,7 @@ namespace SixLabors.ImageSharp /// /// - /// Wraps an existing contiguous memory area of 'width' x 'height' pixels allowing viewing/manipulation as + /// Wraps an existing contiguous memory area of at least 'width' x 'height' pixels allowing viewing/manipulation as /// an instance. /// /// diff --git a/tests/ImageSharp.Tests/Image/ImageTests.WrapMemory.cs b/tests/ImageSharp.Tests/Image/ImageTests.WrapMemory.cs index 02a8381808..16d0baff39 100644 --- a/tests/ImageSharp.Tests/Image/ImageTests.WrapMemory.cs +++ b/tests/ImageSharp.Tests/Image/ImageTests.WrapMemory.cs @@ -355,8 +355,6 @@ namespace SixLabors.ImageSharp.Tests [Theory] [InlineData(0, 5, 5)] [InlineData(20, 5, 5)] - [InlineData(26, 5, 5)] - [InlineData(2, 1, 1)] [InlineData(1023, 32, 32)] public void WrapMemory_MemoryOfT_InvalidSize(int size, int height, int width) { @@ -366,6 +364,20 @@ namespace SixLabors.ImageSharp.Tests Assert.Throws(() => Image.WrapMemory(memory, height, width)); } + [Theory] + [InlineData(25, 5, 5)] + [InlineData(26, 5, 5)] + [InlineData(2, 1, 1)] + [InlineData(1024, 32, 32)] + [InlineData(2048, 32, 32)] + public void WrapMemory_MemoryOfT_ValidSize(int size, int height, int width) + { + var array = new Rgba32[size]; + var memory = new Memory(array); + + Image.WrapMemory(memory, height, width); + } + private class TestMemoryOwner : IMemoryOwner { public Memory Memory { get; set; } @@ -378,8 +390,6 @@ namespace SixLabors.ImageSharp.Tests [Theory] [InlineData(0, 5, 5)] [InlineData(20, 5, 5)] - [InlineData(26, 5, 5)] - [InlineData(2, 1, 1)] [InlineData(1023, 32, 32)] public void WrapMemory_IMemoryOwnerOfT_InvalidSize(int size, int height, int width) { @@ -390,10 +400,22 @@ namespace SixLabors.ImageSharp.Tests } [Theory] - [InlineData(0, 5, 5)] - [InlineData(20, 5, 5)] + [InlineData(25, 5, 5)] [InlineData(26, 5, 5)] [InlineData(2, 1, 1)] + [InlineData(1024, 32, 32)] + [InlineData(2048, 32, 32)] + public void WrapMemory_IMemoryOwnerOfT_ValidSize(int size, int height, int width) + { + var array = new Rgba32[size]; + var memory = new TestMemoryOwner { Memory = array }; + + Image.WrapMemory(memory, height, width); + } + + [Theory] + [InlineData(0, 5, 5)] + [InlineData(20, 5, 5)] [InlineData(1023, 32, 32)] public void WrapMemory_MemoryOfByte_InvalidSize(int size, int height, int width) { @@ -403,6 +425,20 @@ namespace SixLabors.ImageSharp.Tests Assert.Throws(() => Image.WrapMemory(memory, height, width)); } + [Theory] + [InlineData(25, 5, 5)] + [InlineData(26, 5, 5)] + [InlineData(2, 1, 1)] + [InlineData(1024, 32, 32)] + [InlineData(2048, 32, 32)] + public void WrapMemory_MemoryOfByte_ValidSize(int size, int height, int width) + { + var array = new byte[size * Unsafe.SizeOf()]; + var memory = new Memory(array); + + Image.WrapMemory(memory, height, width); + } + [Theory] [InlineData(0, 5, 5)] [InlineData(20, 5, 5)] From fd22d81c7661515c7c7af977b4e9c4ee59ef740b Mon Sep 17 00:00:00 2001 From: Stefan Nikolei Date: Mon, 1 Feb 2021 06:42:06 +0100 Subject: [PATCH 08/10] update submodule shared-infrastructure --- shared-infrastructure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/shared-infrastructure b/shared-infrastructure index b7b9a2755e..af824b8bad 160000 --- a/shared-infrastructure +++ b/shared-infrastructure @@ -1 +1 @@ -Subproject commit b7b9a2755e456a96acbf103494228226d92eddf3 +Subproject commit af824b8bad7307b735662e3eeab39d401c41741c From cf63cd22186707fdc2e2474982dba50ab3a1d6f0 Mon Sep 17 00:00:00 2001 From: Stefan Nikolei Date: Mon, 1 Feb 2021 06:42:52 +0100 Subject: [PATCH 09/10] use when and otherwise in csproj --- src/ImageSharp/ImageSharp.csproj | 19 ++++++++++++------- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/src/ImageSharp/ImageSharp.csproj b/src/ImageSharp/ImageSharp.csproj index e0da2abf0d..6a7f7a600d 100644 --- a/src/ImageSharp/ImageSharp.csproj +++ b/src/ImageSharp/ImageSharp.csproj @@ -14,13 +14,18 @@ A new, fully featured, fully managed, cross-platform, 2D graphics API for .NET - - netcoreapp3.1;netcoreapp2.1;netstandard2.1;netstandard2.0;netstandard1.3;net472 - - - - net5.0;netcoreapp3.1;netcoreapp2.1;netstandard2.1;netstandard2.0;netstandard1.3;net472 - + + + + net5.0;netcoreapp3.1;netcoreapp2.1;netstandard2.1;netstandard2.0;netstandard1.3;net472 + + + + + netcoreapp3.1;netcoreapp2.1;netstandard2.1;netstandard2.0;netstandard1.3;net472 + + + From a060d5c37e9c1f83a26186d62749d58a8fd72a01 Mon Sep 17 00:00:00 2001 From: Stefan Nikolei Date: Mon, 1 Feb 2021 07:36:05 +0100 Subject: [PATCH 10/10] USE SIXLABORS_TESTING instead of CI Env var --- .github/workflows/build-and-test.yml | 4 +++- src/ImageSharp/ImageSharp.csproj | 2 +- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/.github/workflows/build-and-test.yml b/.github/workflows/build-and-test.yml index 4da7813a8a..67259c04f7 100644 --- a/.github/workflows/build-and-test.yml +++ b/.github/workflows/build-and-test.yml @@ -79,12 +79,14 @@ jobs: - name: Build shell: pwsh run: ./ci-build.ps1 "${{matrix.options.framework}}" + env: + SIXLABORS_TESTING: True - name: Test shell: pwsh run: ./ci-test.ps1 "${{matrix.options.os}}" "${{matrix.options.framework}}" "${{matrix.options.runtime}}" "${{matrix.options.codecov}}" env: - CI: True + SIXLABORS_TESTING: True XUNIT_PATH: .\tests\ImageSharp.Tests # Required for xunit - name: Export Failed Output diff --git a/src/ImageSharp/ImageSharp.csproj b/src/ImageSharp/ImageSharp.csproj index 6a7f7a600d..ca6ca16898 100644 --- a/src/ImageSharp/ImageSharp.csproj +++ b/src/ImageSharp/ImageSharp.csproj @@ -15,7 +15,7 @@ - + net5.0;netcoreapp3.1;netcoreapp2.1;netstandard2.1;netstandard2.0;netstandard1.3;net472