Skip to content
Snippets Groups Projects

Image annihilation test fixes

Merged Daniel Nemergut requested to merge annihilator_pex_fix into main

Found a few things while testing the image annihilation this week. These fixes were tested by building the pex and uploading it to dev, so this MR isn't necessary to finish testing.

  • The pex didn't build correctly without pinning the version of greenlet that we use elsewhere (or maybe it only built correctly from my mac, deploying this will tell).
  • The time formatting was incorrect.
  • Added distinct clauses to stop annihilator from checking the same directory multiple times.
  • Future proofing workflow names with variables in the queries.

Merge request reports

Merge request pipeline #15074 passed

Merge request pipeline passed for 642a3c4a

Approval is optional
Ready to merge by members who can write to the target branch.

Merge details

  • 1 commit and 1 merge commit will be added to 2.8.3.1-patch.
  • Source branch will be deleted.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading