Skip to content

Fixing constraints#1618

Open
edwin-snowplow wants to merge 1 commit intomainfrom
edwin-snowplow-patch-1
Open

Fixing constraints#1618
edwin-snowplow wants to merge 1 commit intomainfrom
edwin-snowplow-patch-1

Conversation

@edwin-snowplow
Copy link

@edwin-snowplow edwin-snowplow commented Feb 6, 2026

Adding 2 required constraints to the list of OPs

What changed?

Adding 2 constraints to the list of Organization Policies in GCP

Why?

We spent a long time figuring this out with Bumble. It would be helpful to have this clear for other customers

Adding 2 required constraints to the list of OPs
@netlify
Copy link

netlify bot commented Feb 6, 2026

Deploy Preview for snowplow-docs ready!

Name Link
🔨 Latest commit e2e530b
🔍 Latest deploy log https://app.netlify.com/projects/snowplow-docs/deploys/69863308ca6988000775bba6
😎 Deploy Preview https://deploy-preview-1618--snowplow-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 12 (🔴 down 12 from production)
Accessibility: 91 (no change from production)
Best Practices: 92 (no change from production)
SEO: 95 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@claude
Copy link

claude bot commented Feb 6, 2026

Style Review

The additions look good overall. Here's feedback on potential improvements:

Minor style issue

Heading format (line 45):
The heading "Ensure that the following constraints are turned off" uses passive voice. Per the style guide, headings should prefer imperative voice.

Suggested change:

### Turn off the following constraints

This is more direct and follows the imperative style used in other headings like "Create a new project" and "Set up required permissions".


The new constraint additions follow the existing list format correctly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants