Skip to content

Conversation

@Sugar-pack
Copy link
Contributor

All Submissions:

  • Have you signed our CLA?
  • Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if there is one).
  • Update docs/release-notes/release-notes.md if your changes should be included in the release notes for the next release.

@Sugar-pack Sugar-pack changed the title Update cleanup-all.yml Fix PE cleaning Dec 1, 2022
Copy link
Collaborator

@igor-karpukhin igor-karpukhin left a comment

Choose a reason for hiding this comment

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

Looks good 👍 One comment regarding error format

Copy link
Contributor

@fabritsius fabritsius left a comment

Choose a reason for hiding this comment

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

Looks Great 👍

Just have one question :)

@Sugar-pack Sugar-pack force-pushed the cleaner-fix branch 2 times, most recently from d1e336a to e6a108e Compare December 1, 2022 09:52
@Sugar-pack Sugar-pack merged commit 5e726a0 into main Dec 1, 2022
@Sugar-pack Sugar-pack deleted the cleaner-fix branch December 1, 2022 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants