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

JS/TS/Python/Ruby: Renames diagnostic query files and tests #15465

Conversation

sidshank
Copy link
Contributor

This PR renames the files relating to the diagnostic query that produces information on the number of files extracted. The files have been renamed from SuccessfullExtractedFiles.* to ExtractedFiles.*. All related tests and test files have been renamed too.

The @tags and @id attributes of the queries have been left untouched, consistent with the @tags and @id for similar queries in other languages.

This commit renames the files relating to the diagnostic query that produces information on the number of files extracted. The files have been renamed from "SuccessfullExtractedFiles.*" to "ExtractedFiles.*". All related tests and test files have been renamed too.

The `@tags` and `@id` attributes of the queries have been left untouched, consistent with the `@tags` and `@id` for similar queries in other languages.
@sidshank sidshank added the no-change-note-required This PR does not need a change note label Jan 29, 2024
@sidshank sidshank marked this pull request as ready for review January 29, 2024 20:39
@sidshank sidshank requested review from a team as code owners January 29, 2024 20:39
Copy link
Contributor

@erik-krogh erik-krogh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

I assume other languages have made a similar migration?

Copy link
Contributor

@yoff yoff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - I like that the names follow the semantics. I understand that changing the ids is a little harder...

@sidshank
Copy link
Contributor Author

LGTM.

I assume other languages have made a similar migration?

@erik-krogh That's right. I have added a backlink to the issue that outlines the recommended naming for attributes and file names.

@sidshank sidshank merged commit f557110 into github:main Jan 30, 2024
27 checks passed
@sidshank sidshank deleted the sidshank/rename-file-name-for-extracted-files-diagnostic branch January 30, 2024 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
JS no-change-note-required This PR does not need a change note Python Ruby
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants