Skip to content
This repository has been archived by the owner on Feb 22, 2020. It is now read-only.

Apply SharedInfrastructure, introduce shared guards #33

Merged
merged 12 commits into from
Aug 31, 2019
Merged

Apply SharedInfrastructure, introduce shared guards #33

merged 12 commits into from
Aug 31, 2019

Conversation

antonfirsov
Copy link
Member

@antonfirsov antonfirsov commented Aug 18, 2019 •
edited by JimBobSquarePants
Loading

Prerequisites

  • I have written a descriptive pull-request title
  • I have verified that there are no overlapping pull-requests open
  • I have verified that I am following matches the existing coding patterns and practice as demonstrated in the repository. These follow strict Stylecop rules 👮.
  • I have provided test coverage for my change (where applicable)

Description

Checklist to finalize this PR:

@antonfirsov antonfirsov changed the title Code Issues 0 Pull requests 1 Actions Projects 0 Wiki Security Insights Share Guard code across projects Share Guard code across projects Aug 18, 2019
This was referenced Aug 18, 2019
Merged
Merged
Merged
@antonfirsov antonfirsov changed the title Share Guard code across projects Share Guard code across repositories Aug 18, 2019
@antonfirsov antonfirsov changed the title Share Guard code across repositories Apply SharedInfrastructure, introduce shared guards Aug 29, 2019
@JimBobSquarePants JimBobSquarePants merged commit 2db3a31 into master Aug 31, 2019
@JimBobSquarePants JimBobSquarePants deleted the af/shared-guard branch August 31, 2019 04:04
antonfirsov pushed a commit to SixLabors/ImageSharp that referenced this pull request Jan 19, 2020
@JimBobSquarePants
[SL.Core] Merge pull request SixLabors/Core#33 from SixLabors/af/shar…
4fc16e1 
…ed-guard

Apply SharedInfrastructure, introduce shared guards
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@JimBobSquarePants JimBobSquarePants JimBobSquarePants approved these changes

@dlemstra dlemstra Awaiting requested review from dlemstra

@tocsoft tocsoft Awaiting requested review from tocsoft

Assignees
No one assigned
Labels
build code quality tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@antonfirsov @JimBobSquarePants