Skip to content
This repository was archived by the owner on Sep 6, 2023. It is now read-only.

Conversation

ronaldbarendse
Copy link
Collaborator

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 practise as demonstrated in the repository. These follow strict Stylecop rules 👮.
  • I have provided test coverage for my change (where applicable)

Description

This removes the (unnecessary) RecursivelyDeleteEmptyDirectories() method and removes empty directories while trimming the files (see #795 (comment)).

Copy link
Owner

@JimBobSquarePants JimBobSquarePants left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@ronaldbarendse ronaldbarendse merged commit de98c15 into develop Jun 30, 2020
@ronaldbarendse ronaldbarendse deleted the bugfix/remove-recursivelydeleteemptydirectories branch June 30, 2020 14:37
@ronaldbarendse
Copy link
Collaborator Author

@JimBobSquarePants Can you release ImageProcessor.Web 4.12.1 to NuGet?

@JimBobSquarePants
Copy link
Owner

Done.
https://www.nuget.org/packages/ImageProcessor.Web/4.12.1

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants