Skip to content

fix(retail): add region tag for Python - Update import user events inline source #13494

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

Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,8 @@

# Import user events into a catalog from inline source using Retail API
#
# [START retail_import_user_events_from_inline_source]

import datetime
import random
import string
Expand Down Expand Up @@ -104,3 +106,5 @@ def import_user_events_from_inline_source():


import_user_events_from_inline_source()

# [END retail_import_user_events_from_inline_source]
Comment on lines +109 to +110
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

To maintain consistency and keep the sample code compact, the blank line before the END region tag should be removed. The tag should immediately follow the last line of the code block it encloses, which makes the scope of the sample clearer.

Suggested change
# [END retail_import_user_events_from_inline_source]
# [END retail_import_user_events_from_inline_source]