Skip to content

Fix the audit mess#1424

Merged
bwateratmsft merged 2 commits intomainfrom
bmw/audit
Mar 31, 2026
Merged

Fix the audit mess#1424
bwateratmsft merged 2 commits intomainfrom
bmw/audit

Conversation

@bwateratmsft
Copy link
Copy Markdown
Contributor

No description provided.

@bwateratmsft bwateratmsft requested a review from a team as a code owner March 31, 2026 16:43
Copilot AI review requested due to automatic review settings March 31, 2026 16:43
@bwateratmsft bwateratmsft enabled auto-merge (squash) March 31, 2026 16:44
Copy link
Copy Markdown
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

This PR appears to focus on cleaning up a mix of logging/audit-related issues: small TypeScript correctness/style tweaks plus a sizable dependency refresh in package-lock.json.

Changes:

  • Adjust HTTP request stringification to append the boxed request header to the existing prefix string.
  • Minor TypeScript cleanups (remove redundant initializers / unnecessary array initialization).
  • Update many dev/tooling dependencies in package-lock.json (e.g., esbuild, eslint ecosystem, api-extractor, typescript-eslint).

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/utils/logging/stringifyHttp.ts Fixes request message construction and a whitespace-only doc comment change.
src/commands/deleteResourceGroup/v2/deleteResourceGroupV2.ts Removes unnecessary initialization of resourceGroupsToDelete.
src/api/compatibility/application/CompatibleApplicationResourceBranchDataProvider.ts Removes redundant = undefined initialization.
src/api/auth/createApiSession/createApiSessionInternal.ts Adds Error cause propagation when session creation fails.
package-lock.json Large dependency/tooling upgrades (eslint stack, api-extractor, esbuild, etc.).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@bwateratmsft bwateratmsft merged commit 4b4fdc2 into main Mar 31, 2026
3 checks passed
@bwateratmsft bwateratmsft deleted the bmw/audit branch March 31, 2026 16:59
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.

4 participants