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

Committed updates for 2/4 region tags that were in violation (on devsite). The remaining two found in Quickstart docs are false positives, awaiting updates from cl/376017137.

@Strykrol I think the checks on quickstart_add_data_one and quickstart_add_data_two are valid.

For example, this page:
https://cloud.google.com/firestore/docs/quickstart-servers#add_data
has a link to

But I may be wrong. Why do you think it's a false positive?

The link has commit hash, so it might be ok, but the fact that the master branch doesn't have the same region tag is still confusing.

For a record, the html element has the following property:

data-region-tag="quickstart_add_data_one"

So I assume the devsite build will break if we remove this region tag now.

@tmatsuo let's talk offline - I'm not seeing what you're seeing, but the source file for that page doesn't appear to me to have these region tags anymore.

Looks like the violations resolved after we manually re-published last week.

One more minor lint issue then we should be good:

nox > flake8 --show-source --builtin=gettext --max-complexity=20 --import-order-style=google --exclude=.nox,.cache,env,lib,generated_pb2,*_pb2.py,*_pb2_grpc.py --ignore=E121,E123,E126,E203,E226,E24,E266,E501,E704,W503,W504,I202 --max-line-length=88 --application-import-names distributed_counters,noxfile,distributed_counters_test,.nox,snippets_test,snippets .
./snippets.py:106:1: E302 expected 2 blank lines, found 1
class City(object):
^
./snippets.py:166:1: E302 expected 2 blank lines, found 1
def add_example_data():
^

Done

Thank you! Merged as crwilcox@ is part of cloud-native-db-dpes