Skip to content

deps: Lodash upgrade to 4.18.1#2882

Open
rkolesnikovDX wants to merge 14 commits into
masterfrom
rokolesnikov-microsoft-research-lodash-upgrade
Open

deps: Lodash upgrade to 4.18.1#2882
rkolesnikovDX wants to merge 14 commits into
masterfrom
rokolesnikov-microsoft-research-lodash-upgrade

Conversation

@rkolesnikovDX

Copy link
Copy Markdown
Collaborator

No description provided.

Roman Kolesnikov added 2 commits July 16, 2026 14:31
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 3aafafb2-5e0b-49a1-b15d-ca56d5dd535e
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 3aafafb2-5e0b-49a1-b15d-ca56d5dd535e
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 3aafafb2-5e0b-49a1-b15d-ca56d5dd535e
webpack.staticData.js calls webpack.publisher.js which previously
always fetched an ARM token via getArmToken(). In CI there is no
AZURE_CLIENT_SECRET set, so it fell back to interactive browser auth
which fails with AuthenticationRequiredError.

The static data build does not need a real ARM token — startup.publish.ts
uses a hardcoded StaticSettingsProvider when NODE_ENV === staticDataEnvironment,
and StaticAuthenticator already handles undefined/empty ARM_TOKEN gracefully.

- Add optional skipArmToken parameter to webpack.publisher.js
- Pass { skipArmToken: true } from webpack.staticData.js
@github-actions

Copy link
Copy Markdown

Accessibility Insights Accessibility Insights Action: All applicable checks passed

  • URLs: 12 URL(s) passed, and 0 were not scannable
  • Rules: 27 check(s) passed, and 25 were not applicable
  • Download the Accessibility Insights artifact to view the detailed results of these checks

This scan used axe-core 4.3.2 with Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/86.0.4240.75 Safari/537.36.

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.

3 participants