[AUTOMATED-PR] Update imslib.min.js dependency#3
Open
github-actions[bot] wants to merge 21 commits into
Open
Conversation
…ets and doc files
github-actions
Bot
force-pushed
the
update-imslib
branch
from
c016abc to
4194fb1
Compare
|
Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
Commits
|
Author
|
This pull request is not passing all required checks. Please see this discussion for information on how to get all checks passing. Inconsistent checks can be manually retried. If a test absolutely can not pass for a good reason, please add a comment with an explanation to the PR. |
JackySun9
pushed a commit
that referenced
this pull request
May 29, 2025
…led." (adobecom#4053) Revert "Gap is seen between promo banner and G-nav when page is scrolled. (#3…" This reverts commit 6bedeae.
JackySun9
pushed a commit
that referenced
this pull request
Oct 9, 2025
* MWPW-179531-179764 Brand Concierge - Update Error and Loading Messages * MWPW-179921-179531 Brand Concierge Styling Config (#3) * remove new message border * remove prompt suggestion border
JackySun9
added a commit
that referenced
this pull request
May 20, 2026
Three rollbacks based on the 29 min regression observed with the 4-shard matrix: 1. matrix back to 3-shard (chrome, ipad, iphone full). With only 6 runners, 4-shard meant 1.5 sites concurrent and merge jobs got starved waiting for capture jobs to free runners. 3-shard fits 2 sites concurrently → better throughput in our pool size. 2. Drop actions/cache@v4 for node_modules in all three workflows. Restore step took ~57 s — not faster than `npm ci` on a warm `actions/setup-node` cache (~25 s), and added a Post-cache upload step at the end. Net no win. 3. Slim the merge job to `npm install --no-save --no-package-lock @aws-sdk/client-s3` instead of full `npm ci`. Merge only needs S3 upload; doesn't need playwright, axios, or anything else. Save ~50 s on the merge job. To make #3 possible, `lib/utils.js`'s playwright-core import is now lazy (loaded inside compareScreenshots() rather than at top of file). merge.js only uses validatePath which doesn't need playwright-core. Also rolled back `__config__.waitStrategy: scroll` on bacom-blog. The scroll dance made iphone diff jump 14/18 → 18/18 (100%) — scroll position appears to materially change mobile content rendering on those URLs. Default `footer` wait is fine for this site. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Update ./libs/deps/imslib.min.js to the latest version
Related Issue
Resolves: NO TICKET - AUTOMATED CREATED PR.
Testing instructions
Test URLs
Acrobat:
BACOM:
CC:
Homepage:
Blog:
Milo: