Skip to content

[WPB-27227] Cleanup PR 5343. - #5354

Merged
supersven merged 8 commits into
developfrom
WPB-27227-cleanup-pr-5343
Jul 27, 2026
Merged

[WPB-27227] Cleanup PR 5343.#5354
supersven merged 8 commits into
developfrom
WPB-27227-cleanup-pr-5343

Conversation

@fisx

@fisx fisx commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

https://wearezeta.atlassian.net/browse/WPB-27227

as promised in #5343 (comment) :)

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Jul 21, 2026
@fisx
fisx marked this pull request as ready for review July 21, 2026 15:20
@fisx
fisx requested review from a team as code owners July 21, 2026 15:20
@@ -0,0 +1 @@
Cleanup PR 5343.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I would delete this file. It provides little value to the reader who will care about the release, not single PRs.

Comment on lines +72 to +73
-- | Delete app. This is meant for calling from when deleting team
-- members and does not check authentication.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
-- | Delete app. This is meant for calling from when deleting team
-- members and does not check authentication.
-- | Delete app. This is meant for calling to delete team
-- members. It does not check authentication.

The sentence reads a bit clunky. 馃槄

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Also I can't parse it any more... :) Thanks!

@supersven supersven left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You may want to consider my comments. However, there's nothing severe enough to block merging.

@supersven
supersven merged commit 3cb74c4 into develop Jul 27, 2026
9 checks passed
@supersven
supersven deleted the WPB-27227-cleanup-pr-5343 branch July 27, 2026 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants