Here is the summary of changes.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

• Refresh this comment

Hey @Avani-Thakker-Crest this file is growing to quite a big number of lines. Would it be possible to slip it into multiple files? So that for example each region tag gets its own file?

It would also allow you to remove some # noqa: comments, as you wouldn't repeat imports etc.

Hey @Avani-Thakker-Crest this file is growing to quite a big number of lines. Would it be possible to slip it into multiple files? So that for example each region tag gets its own file?

It would also allow you to remove some # noqa: comments, as you wouldn't repeat imports etc.

Thank you @m-strzelczyk for your comment. Definitely, we can split into multiple files. Please let me know how should I split:

1. Split deid related samples into 3-4 logical files. I had already split deid samples into two files in the past a) deid_table.py containing all samples deidentifying table data and b) deid.py containing the remaining samples. We can further go ahead and split into 2 more files.
2. Split samples such a way that one file corresponds to one sample. This would result in entire restructuring of all the dlp samples, be it inspect.py, risk.py and so on to maintain uniformity.
3. For now, we can create a separate file for deidentify_cloud_storage sample and a corresponding test file.

Please let me know which option would be appropriate to perform. Thank you!

Hey @Avani-Thakker-Crest this file is growing to quite a big number of lines. Would it be possible to slip it into multiple files? So that for example each region tag gets its own file?
It would also allow you to remove some # noqa: comments, as you wouldn't repeat imports etc.

Thank you @m-strzelczyk for your comment. Definitely, we can split into multiple files. Please let me know how should I split:

1. Split deid related samples into 3-4 logical files. I had already split deid samples into two files in the past a) deid_table.py containing all samples deidentifying table data and b) deid.py containing the remaining samples. We can further go ahead and split into 2 more files.
2. Split samples such a way that one file corresponds to one sample. This would result in entire restructuring of all the dlp samples, be it inspect.py, risk.py and so on to maintain uniformity.
3. For now, we can create a separate file for deidentify_cloud_storage sample and a corresponding test file.

Please let me know which option would be appropriate to perform. Thank you!

Hi @m-strzelczyk , for now I have created new files for this particular sample. And for the rest samples, I have created a backlog issue #10601 .