-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Closed
Labels
App ConfigurationAzure.ApplicationModel.ConfigurationAzure.ApplicationModel.ConfigurationClientThis issue points to a problem in the data-plane of the library.This issue points to a problem in the data-plane of the library.help wantedThis issue is tracking work for which community contributions would be welcomed and appreciatedThis issue is tracking work for which community contributions would be welcomed and appreciated
Milestone
Description
Fix lint errors found in app-configuration package by ESLint. Following are the steps to run ESLint for appconfiguration package and reproduce this issue.
- Set up your dev environment if not already done so as mentioned here
- Go to
<repo root>/sdk/appconfiguration/app-configuration
- run
rushx lint
- Command in step 2) generates an html report in directory
<repo root>/sdk/appconfiguration/app-configuration
with name ends with lintReport.html - All lint errors found in this package is listed on html report.
Once all known issues are resolved, below change is required in package.json
file in package root <repo root>/sdk/appconfiguration/app-configuration
to treat any new lint regression as hard failure in CI.
- Remove following snippet from lint command in
package.json
-f html -o template-lintReport.html || exit 0
Note: HTML report name prefix may be different for each package name to differentiate the report for each package.
richardpark-msft
Metadata
Metadata
Assignees
Labels
App ConfigurationAzure.ApplicationModel.ConfigurationAzure.ApplicationModel.ConfigurationClientThis issue points to a problem in the data-plane of the library.This issue points to a problem in the data-plane of the library.help wantedThis issue is tracking work for which community contributions would be welcomed and appreciatedThis issue is tracking work for which community contributions would be welcomed and appreciated