Skip to content

update readme to fix Lint Diff Error #30340

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
xuhumsft merged 6 commits into from
Aug 29, 2024
Merged

update readme to fix Lint Diff Error #30340

Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
dc0cc0e
update readme to fix Lint Diff Error
xuhumsft Aug 27, 2024
8dbd296
fix format
xuhumsft Aug 28, 2024
d6842b5
remove ----
xuhumsft Aug 28, 2024
ee5f076
fix line break
xuhumsft Aug 29, 2024
9f3fffc
Merge branch 'dev-Sentinel-securityinsights-Microsoft.SecurityInsight…
xuhumsft Aug 29, 2024
50a7930
Revert "remove ----"
xuhumsft Aug 29, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 8 additions & 7 deletions specification/securityinsights/resource-manager/readme.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,8 @@ This is the AutoRest configuration file for SecurityInsights.

## Getting Started

To build the SDK for SecurityInsights, simply [Install AutoRest](https://aka.ms/autorest/install) and in this folder, run:
To build the SDK for Azure Machine Learning, simply [Install AutoRest](https://aka.ms/autorest/install) and
in this folder, run:

> `autorest`

Expand All @@ -31,7 +32,6 @@ tag: package-preview-2024-04

---


### Tag: package-preview-2024-04

These settings apply only when `--tag=package-preview-2024-04` is specified on the command line.
Expand Down Expand Up @@ -101,16 +101,17 @@ suppressions:
- code: AvoidAdditionalProperties
from: ThreatIntelligenceQuery.json
reason: These properties are required in current API. The team is working on a new version of API to resolve it in the future release.
- code: GetCollectionOnlyHasValueAndNextLink
- code: GetCollectionOnlyHasValueAndNextLink
from: Entities.json
reason: This API is published to customers and we have not changed it in the past year, nor will we be able to change it without breaking changes to customers.
- code: DefinitionsPropertiesNamesCamelCase
- code: DefinitionsPropertiesNamesCamelCase
from: Entities.json
reason: This API is published to customers and we have not changed it in the past year, nor will we be able to change it without breaking changes to customers.
- code: PutRequestResponseSchemeArm
- code: PutRequestResponseSchemeArm
from: Entities.json
reason: This API is published to customers and we have not changed it in the past year, nor will we be able to change it without breaking changes to customers.
```

### Tag: package-2024-03

These settings apply only when `--tag=package-2024-03` is specified on the command line.
Expand Down Expand Up @@ -1040,9 +1041,9 @@ directive:
- suppress: R4017
reason: Our resources do not support list by subscription. They're not top-level resources. To get a SecurityInsights resource, we should have a subscription as well as a resource group and Log Analytics workspace.
- suppress: OBJECT_ADDITIONAL_PROPERTIES
reason: 'Caused by a duplicate Resource definition in our common directory that contains systemData. We were instructed to supress this by Swagger reviewr.'
reason: 'Caused by a duplicate Resource definition in our common directory that contains systemData. We were instructed to suppress this by Swagger reviewer.'
- suppress: GuidUsage
reason: 'Tenent Id should be a Guid'
reason: 'Tenant Id should be a Guid'
```

---
Expand Down