Skip to content

[Editor] Don't save the image when updating a stamp annotation#20233

Merged
calixteman merged 1 commit intomozilla:masterfrom
calixteman:dont_add_image_stamp_update
Sep 4, 2025
Merged

[Editor] Don't save the image when updating a stamp annotation#20233
calixteman merged 1 commit intomozilla:masterfrom
calixteman:dont_add_image_stamp_update

Conversation

@calixteman
Copy link
Contributor

It adds some useless bytes in the file.

@timvandermeij
Copy link
Contributor

/botio test

@moz-tools-bot
Copy link
Collaborator

From: Bot.io (Linux m4)


Received

Command cmd_test from @timvandermeij received. Current queue size: 0

Live output at: http://54.241.84.105:8877/3b943a7480c3037/output.txt

@moz-tools-bot
Copy link
Collaborator

From: Bot.io (Windows)


Received

Command cmd_test from @timvandermeij received. Current queue size: 0

Live output at: http://54.193.163.58:8877/21848c07aac9c2f/output.txt

Copy link
Contributor

@timvandermeij timvandermeij left a comment

Choose a reason for hiding this comment

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

r=me, with passing tests. Thanks!

@moz-tools-bot
Copy link
Collaborator

From: Bot.io (Linux m4)


Success

Full output at http://54.241.84.105:8877/3b943a7480c3037/output.txt

Total script time: 37.44 mins

  • Unit tests: Passed
  • Integration Tests: Passed
  • Regression tests: Passed

@moz-tools-bot
Copy link
Collaborator

From: Bot.io (Windows)


Failed

Full output at http://54.193.163.58:8877/21848c07aac9c2f/output.txt

Total script time: 74.78 mins

  • Unit tests: FAILED
  • Integration Tests: Passed
  • Regression tests: Passed

@calixteman calixteman merged commit 7b87c22 into mozilla:master Sep 4, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants