Skip to content

Fix all lint errors#1356

Merged
alexweininger merged 2 commits intomainfrom
alex/lint
Feb 4, 2026
Merged

Fix all lint errors#1356
alexweininger merged 2 commits intomainfrom
alex/lint

Conversation

@alexweininger
Copy link
Member

No description provided.

@alexweininger alexweininger requested a review from a team as a code owner February 4, 2026 14:48
Copilot AI review requested due to automatic review settings February 4, 2026 14:48
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

This PR primarily adds standardized Microsoft copyright/license headers to previously unannotated files and aligns a couple of existing headers with the prevailing License.txt convention, while also tweaking ESLint configuration to ignore generated API output. These changes appear aimed at resolving lint rules relating to license headers and unnecessary linting of build artifacts.

Changes:

  • Add the standard Microsoft MIT license header block to a set of test files, API utility files, and core extension source files that previously lacked it.
  • Normalize existing headers in tenant and activity-log branch data provider files to match the License.txt wording and style used elsewhere in the repo.
  • Update eslint.config.mjs to ignore api/dist/** and api/out/**, preventing linting of compiled output.

Reviewed changes

Copilot reviewed 23 out of 23 changed files in this pull request and generated no comments.

Show a summary per file
File Description
test/wrapFunctionsInTelemetry.test.ts Adds the standard license header to the telemetry wrapping test suite.
test/grouping.test.ts Adds the standard license header to grouping tests.
test/api/workspaceResourceProvider.test.ts Adds the standard license header to workspace resource provider API tests.
test/api/viewProperties.test.ts Adds the standard license header to view properties API tests.
test/api/v1.test.ts Adds the standard license header to v1 API compatibility tests.
test/api/subscription.test.ts Adds the standard license header to subscription-related API tests.
test/api/hasPortalUrl.test.ts Adds the standard license header to hasPortalUrl API tests.
test/api/getAzureResourcesExtensionApi.test.ts Adds the standard license header to tests for obtaining the Azure Resources extension API.
test/api/branchDataProvider.test.ts Adds the standard license header to branch data provider API tests.
test/api/azureResourceBranchDataProvider.test.ts Adds the standard license header to Azure resource branch data provider tests.
test/api/api.ts Adds the standard license header to the shared API test helper module.
test/api/TestBranchDataProvider.ts Adds the standard license header to the test branch data provider implementation used in API tests.
test/ResourceGroupGroupingItem.test.ts Adds the standard license header to resource group grouping item tests.
src/utils/logging/nodeFetch/nodeFetch.ts Adds the standard license header to the Node fetch logging utility without changing its behavior.
src/utils/activityUtils.ts Adds the standard license header to activity utility functions.
src/tree/tenants/TenantDefaultBranchDataProvider.ts Normalizes the license header to the License.txt wording and formatting and adds a blank line after the header.
src/tree/azure/idPrefix.ts Adds the standard license header to the Azure ID prefix utility.
src/tree/activityLog/ActivityLogDefaultBranchDataProvider.ts Normalizes the license header to the License.txt wording and formatting and adds a blank line after the header.
src/services/getSubscriptionProviderFactory.ts Adds the standard license header to the subscription provider factory service.
src/services/AzureResourcesService.ts Adds the standard license header to the Azure resources service implementation.
src/cloudConsole/cloudConsoleUtils.ts Adds the standard license header to cloud console utility functions.
eslint.config.mjs Extends ESLint configuration to ignore api/dist/** and api/out/** so that generated API artifacts are not linted.
api/src/utils/wrapper.ts Adds the standard license header to the API wrapper utility while leaving the wrapper logic unchanged.

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

@alexweininger alexweininger enabled auto-merge (squash) February 4, 2026 15:28
@alexweininger alexweininger merged commit 008003c into main Feb 4, 2026
3 checks passed
@alexweininger alexweininger deleted the alex/lint branch February 4, 2026 17:42
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