Skip to content

Suppress 100+ baseline-browser-mapping warnings in CI#26544

Open
frankmueller-msft wants to merge 1 commit intomicrosoft:mainfrom
frankmueller-msft:fix/suppress-baseline-browser-mapping-warnings
Open

Suppress 100+ baseline-browser-mapping warnings in CI#26544
frankmueller-msft wants to merge 1 commit intomicrosoft:mainfrom
frankmueller-msft:fix/suppress-baseline-browser-mapping-warnings

Conversation

@frankmueller-msft
Copy link
Contributor

@frankmueller-msft frankmueller-msft commented Feb 25, 2026

Summary

  • Adds BASELINE_BROWSER_MAPPING_IGNORE_OLD_DATA=1 to both the build and lint steps in include-build-lint.yml
  • Eliminates 100+ identical warnings ("data is over two months old") from baseline-browser-mapping, a transitive dependency of browserslist that fires once per package during builds
  • The env var is the upstream-supported suppression mechanism

Note

This suppresses the warning in CI only — local builds still surface it so developers know the data is stale. A follow-up should add a Renovate rule or periodic npx update-browserslist-db to keep the data current rather than permanently silencing the staleness signal.

Test plan

  • repo-policy-check passes
  • Verify CI build logs no longer contain "data is over two months old" warnings (requires triggering Build - client packages pipeline, which only runs when client code is changed)

🤖 Generated with Claude Code

Copilot AI review requested due to automatic review settings February 25, 2026 21:49
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the shared Azure Pipelines lint/build template to reduce CI log noise from baseline-browser-mapping warnings emitted via browserslist during ESLint runs.

Changes:

  • Adds BASELINE_BROWSER_MAPPING_IGNORE_OLD_DATA=1 to the npm run ci:build (template taskBuild) step environment in include-build-lint.yml.

Comment on lines 30 to 33
# Suppress "data is over two months old" warnings from baseline-browser-mapping,
# a transitive dependency of browserslist. Without this, the warning fires once
# per package during ESLint, producing 100+ identical warnings in build logs.
BASELINE_BROWSER_MAPPING_IGNORE_OLD_DATA: 1
Copy link

Copilot AI Feb 25, 2026

Choose a reason for hiding this comment

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

BASELINE_BROWSER_MAPPING_IGNORE_OLD_DATA is only being set for the npm run ${{ parameters.taskBuild }} step. When parameters.taskLint is true (many pipelines still run a separate npm run lint), the lint task will continue to emit the baseline-browser-mapping warning. Consider also setting this env var on the lint Npm@1 task (or moving it to a higher scope variable/env shared by both tasks) so the log suppression is consistent.

Copilot uses AI. Check for mistakes.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Addressed — the env var is now set on both the build and lint steps.

@frankmueller-msft frankmueller-msft force-pushed the fix/suppress-baseline-browser-mapping-warnings branch from d4989a0 to 75d899d Compare February 26, 2026 00:12
Add BASELINE_BROWSER_MAPPING_IGNORE_OLD_DATA=1 to both the build and
lint steps in include-build-lint.yml. This eliminates ~128 identical
"data is over two months old" warnings from baseline-browser-mapping,
a transitive dependency of browserslist that fires once per package.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@frankmueller-msft frankmueller-msft force-pushed the fix/suppress-baseline-browser-mapping-warnings branch from 75d899d to 52bba8b Compare February 26, 2026 04:21
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.

2 participants