Skip to content
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

Jump to bottom

Swagger LintDiff false positives for EvenSegmentedPathForPutOperation #7801

Open
cataggar opened this issue Mar 1, 2024 · 1 comment
Open

Swagger LintDiff false positives for EvenSegmentedPathForPutOperation #7801

cataggar opened this issue Mar 1, 2024 · 1 comment
Assignees
@konrad-jamrozik
Labels
Central-EngSys This issue is owned by the Engineering System team. Spec PR Tools Tooling that runs in azure-rest-api-specs repo.

Comments

@cataggar
Copy link
Member

cataggar commented Mar 1, 2024

There look to be 8 false positives for EventSegmentedPathForPutOperation in this PR check:
https://github.com/Azure/azure-rest-api-specs/pull/28023/checks?check_run_id=22174134612

It is using Swagger LintDiff via openapi-validator v2.2.0.

It looks like that rule was modified on January 8th by @rkmanda.
Azure/azure-openapi-validator#632

That repo is lacking git tags, so I'm not sure if the version in use contains those changes.
@github-actions github-actions bot added the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Mar 1, 2024
@cataggar cataggar mentioned this issue Mar 1, 2024
Merged
7 tasks
@cataggar cataggar mentioned this issue Mar 12, 2024
Merged
7 tasks
cataggar added a commit to cataggar/azure-rest-api-specs that referenced this issue Mar 15, 2024
@cataggar
EvenSegmentedPathForPutOperation false positives. Azure/azure-sdk-too…
75264b5 
…ls#7801
@mikeharder
Copy link
Member

I believe this is a dup of Azure/azure-openapi-validator#646

@kurtzeborn kurtzeborn added Central-EngSys This issue is owned by the Engineering System team. Spec PR Tools Tooling that runs in azure-rest-api-specs repo. and removed needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. labels Jun 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@konrad-jamrozik konrad-jamrozik

Labels
Central-EngSys This issue is owned by the Engineering System team. Spec PR Tools Tooling that runs in azure-rest-api-specs repo.
Projects
Azure SDK EngSys 📈
Status: 📋 Backlog
Spec PR Tools
Status: 📋 Backlog
Milestone
No milestone
Development

No branches or pull requests
4 participants
@konrad-jamrozik @mikeharder @kurtzeborn @cataggar