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.

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

[BUG] two copies of PiiDetectionSkillMaskingMode.cs varying only by case are present #23320

Closed
idg10 opened this issue Aug 13, 2021 · 2 comments · Fixed by #23326
Closed

[BUG] two copies of PiiDetectionSkillMaskingMode.cs varying only by case are present #23320

idg10 opened this issue Aug 13, 2021 · 2 comments · Fixed by #23326
Assignees
Labels
Client This issue points to a problem in the data-plane of the library. customer-reported Issues that are reported by GitHub users external to the Azure organization. needs-team-attention Workflow: This issue needs attention from Azure service team or SDK team question The issue doesn't require a change to the product in order to be resolved. Most issues start as that Search

Comments

@idg10
Copy link
Contributor

idg10 commented Aug 13, 2021

Describe the bug
As of 2789b52 the repo contains both this:

sdk/search/Azure.Search.Documents/src/Generated/Models/PIIDetectionSkillMaskingMode.cs

and this:

sdk/search/Azure.Search.Documents/src/Generated/Models/PiiDetectionSkillMaskingMode.cs

Note that the first three letters of the file vary only by case. (PII vs Pii.)

It looks like after updating Autorest, the name of this generated file changed the casing, but I believe these are logically meant to be the same file.

Expected behavior
There should be just one copy of this file, probably cased as PiiDetectionSkillMaskingMode.cs (i.e., the PIIDetectionSkillMaskingMode.cs copy should have been deleted when the one starting Pii was added).

Actual behavior (include Exception or Stack Trace)
Two copies of the file with different casing are present.

To Reproduce
This is most easily seen by cloning the repo in Windows, which will produce this warning:

warning: the following paths have collided (e.g. case-sensitive paths
on a case-insensitive filesystem) and only one from the same
colliding group is in the working tree:

  'sdk/search/Azure.Search.Documents/src/Generated/Models/PIIDetectionSkillMaskingMode.cs'
  'sdk/search/Azure.Search.Documents/src/Generated/Models/PiiDetectionSkillMaskingMode.cs'

Environment:
GitHub. (The problem is in the repo.)

@ghost ghost added needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. customer-reported Issues that are reported by GitHub users external to the Azure organization. question The issue doesn't require a change to the product in order to be resolved. Most issues start as that labels Aug 13, 2021
@jsquire jsquire added Client This issue points to a problem in the data-plane of the library. needs-team-attention Workflow: This issue needs attention from Azure service team or SDK team Search labels Aug 13, 2021
@ghost ghost removed the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Aug 13, 2021
@jsquire
Copy link
Member

jsquire commented Aug 13, 2021

//fyi: @christothes

@jsquire
Copy link
Member

jsquire commented Aug 13, 2021

Thank you for your feedback. Tagging and routing to the team member best able to assist.

@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Client This issue points to a problem in the data-plane of the library. customer-reported Issues that are reported by GitHub users external to the Azure organization. needs-team-attention Workflow: This issue needs attention from Azure service team or SDK team question The issue doesn't require a change to the product in order to be resolved. Most issues start as that Search
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants