feat: article, googleworkspace module and managing #66
Conversation
✅ Deploy Preview for masterpoint ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
## Walkthrough
The changes update the `.gitignore` file to add ignore patterns for `.cursor/`, `CLAUDE.md`, and `.claude/`, with explanatory comments. Additionally, a new blog post is introduced that discusses managing Google Workspace users via Terraform, providing practical examples, module design decisions, and implementation details.
## Changes
| File(s) | Change Summary |
|-----------------------------------------------------------------------------------------|-----------------------------------------------------------------------------------------------------------------|
| .gitignore | Added ignore patterns for `.cursor/`, `CLAUDE.md`, and `.claude/` with comments about developer tooling configs. |
| content/blog/platform-engineering-way-to-manage-google-workspace-users.md | Added a new blog post detailing Google Workspace user management with Terraform, including examples and design rationale. |
## Sequence Diagram(s)
```mermaid
sequenceDiagram
participant Developer
participant GitHub
participant Terraform Module
participant Google Workspace
Developer->>GitHub: Submit PR with user/group config changes
GitHub->>Terraform Module: Run validation/integration tests
Terraform Module->>Google Workspace: Apply user/group updates via provider
Google Workspace-->>Terraform Module: Confirm changes
Terraform Module-->>GitHub: Report test/apply results
GitHub-->>Developer: Notify PR statusSuggested reviewers
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (7)
.gitignore (1)
6-7: Confirm intended ignore pattern for CLAUDE.md
The entryCLAUDE.mdwill ignore files with that exact name anywhere in the repository. If you only intend to ignore the root-level file, consider prefixing it with a slash (e.g.,/CLAUDE.md).content/blog/manaing-google-workspace-with-terrafrom.md (6)
2-3: Update draft status before publishing
You’ve setdraft: true; remember to switch this todraft: falsewhen ready to go live so the article becomes visible on the site.
43-46: Hyphenate compound adjectives
Consider hyphenating “open source” and “command line” as “open-source” and “command-line” for consistency and improved readability.
51-51: Use “sign in to” instead of “sign into”
Replace “sign into multiple AWS accounts” with “sign in to multiple AWS accounts” to reflect standard usage.
52-52: Correct compound word spelling
Change “straight forward” to “straightforward” to match standard English usage.
152-152: Consistent ordered list numbering
Markdown lint rule MD029 prefers all ordered items start with “1.”. Change the “2.” at this line to “1.” to satisfy consistent numbering.
1-293: Run automatic formatting
The file fails Prettier checks. Please runtrunk fmtorprettier --writeto ensure consistent Markdown formatting before merging.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
.gitignore(1 hunks)content/blog/manaing-google-workspace-with-terrafrom.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
content/blog/manaing-google-workspace-with-terrafrom.md
[grammar] ~34-~34: ‘a’ may be redundant when used with the uncountable noun ‘permission’.
Context: ...ategy. * Nobody remembers why or when a permission was changed. * Group membership becom...
(A_UNCOUNTABLE_NOUN)
[typographical] ~39-~39: Except for inverted sentences, ‘dare we’ requires a question mark at the end of the sentence.
Context: ... manage across many different SaaS tools. It's important to have systems in place...
(MD_PRP_QUESTION_MARK)
[uncategorized] ~43-~43: If this is a compound adjective that modifies the following noun, use a hyphen.
Context: ...ith Terraform We looked at a couple of open source solutions and decided on using the Terr...
(EN_COMPOUND_ADJECTIVE_INTERNAL)
[style] ~44-~44: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...ervices, so this was an easy decision. We heard great things about [GAM](https://...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
[uncategorized] ~51-~51: Possible missing comma found.
Context: ...m's permissions. 3. Easily manage SSO configurations allowing engineers to sign into multipl...
(AI_HYDRA_LEO_MISSING_COMMA)
[grammar] ~51-~51: The verb ‘sign into’ is not standard English, except in the context of the law (“The bill was signed into law”). Write “sign in to”. For websites and computers, other options are “log in to” or “log on to”.
Context: ...SO configurations allowing engineers to sign into multiple AWS accounts using their Googl...
(SIGN_INTO)
[uncategorized] ~52-~52: The adjective or adverb “straightforward” (= easy, precise) is spelled as one word.
Context: ...e user and team settings in a clear and straight forward approach. Here's the GitHub link for t...
(STRAIGHT_FORWARD_COMPOUND)
[uncategorized] ~106-~106: Possible missing comma found.
Context: ... Design Decisions to Make It Intuitive Below we point out a few Terraform module des...
(AI_HYDRA_LEO_MISSING_COMMA)
[uncategorized] ~287-~287: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short).
Context: ... care of the rest. We love this workflow and we hope it's useful to you and your tea...
(COMMA_COMPOUND_SENTENCE)
🪛 markdownlint-cli2 (0.17.2)
content/blog/manaing-google-workspace-with-terrafrom.md
152-152: Ordered list item prefix
Expected: 1; Actual: 2; Style: 1/1/1
(MD029, ol-prefix)
🪛 GitHub Check: Trunk Check
content/blog/manaing-google-workspace-with-terrafrom.md
[notice] 152-152: markdownlint(MD029)
[new] Ordered list item prefix
[failure] 1-1: prettier
Incorrect formatting, autoformat by running 'trunk fmt'
⏰ Context from checks skipped due to timeout of 90000ms (3)
- GitHub Check: Redirect rules - masterpoint
- GitHub Check: Header rules - masterpoint
- GitHub Check: Pages changed - masterpoint
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (8)
content/blog/managing-google-workspace-with-terraform.md (8)
10-10: Replace raw HTML callout with Markdown blockquote
Raw HTML reduces readability and bypasses Markdown styling. Use a>blockquote with Markdown for the callout section instead of an inline<p>tag.
35-35: Remove indefinite article before “permission”
“permission” is uncountable here; consider “when permissions were changed” or “when permission changes occurred.”
40-40: Comma parenthetical phrase
Insert commas around “and dare we say departments” for proper punctuation and readability:…teams, (and dare we say, departments), become more distributed…
44-44: Hyphenate “open-source”
Use the compound adjective “open-source solutions” to align with style guidelines.
44-46: Vary opening sentence structure
Multiple consecutive sentences begin with “We.” Consider rewording one of these to improve flow and avoid repetition.
52-53: Clarify list items and fix modifiers
- Add a comma before “allowing” for clarity.
- Change “sign into” → “sign in to” when referring to logging in.
- Merge “straight forward” → “straightforward” for the compound adjective.
111-111: Comma after introductory phrase
Insert a comma after “Below” for correct punctuation:Below, we point out a few…
291-291: Add comma in compound sentence
Consider adding a comma before “and” to separate independent clauses:We love this workflow, and we hope it’s useful…
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
.cursor/rules/blog-article-review.mdc(1 hunks)content/blog/managing-google-workspace-with-terraform.md(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- .cursor/rules/blog-article-review.mdc
🧰 Additional context used
🪛 LanguageTool
content/blog/managing-google-workspace-with-terraform.md
[grammar] ~35-~35: ‘a’ may be redundant when used with the uncountable noun ‘permission’.
Context: ...trategy. - Nobody remembers why or when a permission was changed. - Group membership becomes...
(A_UNCOUNTABLE_NOUN)
[typographical] ~40-~40: Except for inverted sentences, ‘dare we’ requires a question mark at the end of the sentence.
Context: ... manage across many different SaaS tools. It's important to have systems in place...
(MD_PRP_QUESTION_MARK)
[uncategorized] ~44-~44: If this is a compound adjective that modifies the following noun, use a hyphen.
Context: ...ith Terraform We looked at a couple of open source solutions and decided on using the Terr...
(EN_COMPOUND_ADJECTIVE_INTERNAL)
[style] ~45-~45: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...ervices, so this was an easy decision. We heard great things about [GAM](https://...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
[uncategorized] ~52-~52: Possible missing comma found.
Context: ...eam's permissions. 3. Easily manage SSO configurations allowing engineers to sign into multipl...
(AI_HYDRA_LEO_MISSING_COMMA)
[grammar] ~52-~52: The verb ‘sign into’ is not standard English, except in the context of the law (“The bill was signed into law”). Write “sign in to”. For websites and computers, other options are “log in to” or “log on to”.
Context: ...SO configurations allowing engineers to sign into multiple AWS accounts using their Googl...
(SIGN_INTO)
[uncategorized] ~53-~53: The adjective or adverb “straightforward” (= easy, precise) is spelled as one word.
Context: ...e user and team settings in a clear and straight forward approach. Here's the GitHub link for t...
(STRAIGHT_FORWARD_COMPOUND)
[uncategorized] ~110-~110: Possible missing comma found.
Context: ... Design Decisions to Make It Intuitive Below we point out a few Terraform module des...
(AI_HYDRA_LEO_MISSING_COMMA)
[uncategorized] ~291-~291: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short).
Context: ... care of the rest. We love this workflow and we hope it's useful to you and your tea...
(COMMA_COMPOUND_SENTENCE)
⏰ Context from checks skipped due to timeout of 90000ms (3)
- GitHub Check: Redirect rules - masterpoint
- GitHub Check: Header rules - masterpoint
- GitHub Check: Pages changed - masterpoint
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (8)
content/blog/managing-google-workspace-with-terraform.md (8)
10-10: Close HTML tags in callout frontmatterThe
calloutopens<p>and<b>tags but doesn’t close them, which can break rendering in some markdown parsers.-callout: <p>👋 <b>If you're ready to take your infrastructure to the next level, we're here to help. We love to work together with engineering teams to help them build well-documented, scalable, automated IaC that make their jobs easier. <a href='/contact'>Get in touch!</a> +callout: <p>👋 <b>If you're ready to take your infrastructure to the next level, we're here to help. We love to work together with engineering teams to help them build well-documented, scalable, automated IaC that make their jobs easier. <a href='/contact'>Get in touch!</a></b></p>
35-35: Remove unnecessary article before “permission”In this context “permission” is uncountable; drop the “a” for clarity.
- - Nobody remembers why or when a permission was changed. + - Nobody remembers why or when permission was changed.
40-40: Set off the parenthetical “dare we say” with commasAdd commas to improve readability around the parenthetical phrase.
- As a company grows past 10, dozens, or 100+ people, teams (and dare we say departments) become more distributed and harder to manage… + As a company grows past 10, dozens, or 100+ people, teams (and, dare we say, departments) become more distributed and harder to manage…
44-44: Hyphenate “open-source” when used as an adjectiveCompound adjectives should be hyphenated.
- We looked at a couple of open source solutions and decided on using the Terraform Google Workspace provider. + We looked at a couple of open-source solutions and decided on using the Terraform Google Workspace provider.
52-53: Insert comma, correct “sign in to”, and merge “straightforward”Improve grammar and compound-word usage in list items.
- 3. Easily manage SSO configurations allowing engineers to sign into multiple AWS accounts using their Google Identity. - 4. Intuitively organize user and team settings in a clear and straight forward approach. + 3. Easily manage SSO configurations, allowing engineers to sign in to multiple AWS accounts using their Google Identity. + 4. Intuitively organize user and team settings in a clear and straightforward approach.
63-63: Remove extra period in parenthesesTwo dots appear after the closing parenthesis.
- …create your own `users.yaml` and `groups.yaml` files).. + …create your own `users.yaml` and `groups.yaml` files).
110-110: Add comma after introductory word “Below”An introductory adverbial phrase should be followed by a comma.
- Below we point out a few Terraform module design decisions we made to reduce friction as others use the module. + Below, we point out a few Terraform module design decisions we made to reduce friction as others use the module.
291-291: Add comma before coordinating conjunction “and”Coordinate independent clauses with a comma.
- We love this workflow and we hope it's useful to you and your team! + We love this workflow, and we hope it's useful to you and your team!
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
content/blog/managing-google-workspace-with-terraform.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
content/blog/managing-google-workspace-with-terraform.md
[grammar] ~35-~35: ‘a’ may be redundant when used with the uncountable noun ‘permission’.
Context: ...trategy. - Nobody remembers why or when a permission was changed. - Group membership becomes...
(A_UNCOUNTABLE_NOUN)
[typographical] ~40-~40: Except for inverted sentences, ‘dare we’ requires a question mark at the end of the sentence.
Context: ... manage across many different SaaS tools. It's important to have systems in place...
(MD_PRP_QUESTION_MARK)
[uncategorized] ~44-~44: If this is a compound adjective that modifies the following noun, use a hyphen.
Context: ...ith Terraform We looked at a couple of open source solutions and decided on using the Terr...
(EN_COMPOUND_ADJECTIVE_INTERNAL)
[style] ~45-~45: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...ervices, so this was an easy decision. We heard great things about [GAM](https://...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
[uncategorized] ~52-~52: Possible missing comma found.
Context: ...eam's permissions. 3. Easily manage SSO configurations allowing engineers to sign into multipl...
(AI_HYDRA_LEO_MISSING_COMMA)
[grammar] ~52-~52: The verb ‘sign into’ is not standard English, except in the context of the law (“The bill was signed into law”). Write “sign in to”. For websites and computers, other options are “log in to” or “log on to”.
Context: ...SO configurations allowing engineers to sign into multiple AWS accounts using their Googl...
(SIGN_INTO)
[uncategorized] ~53-~53: The adjective or adverb “straightforward” (= easy, precise) is spelled as one word.
Context: ...e user and team settings in a clear and straight forward approach. Here's the GitHub link for t...
(STRAIGHT_FORWARD_COMPOUND)
[typographical] ~63-~63: Two consecutive dots
Context: ...wn users.yaml and groups.yaml files).. 2. Create New Resources Start...
(DOUBLE_PUNCTUATION)
[uncategorized] ~110-~110: Possible missing comma found.
Context: ... Design Decisions to Make It Intuitive Below we point out a few Terraform module des...
(AI_HYDRA_LEO_MISSING_COMMA)
[uncategorized] ~291-~291: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short).
Context: ... care of the rest. We love this workflow and we hope it's useful to you and your tea...
(COMMA_COMPOUND_SENTENCE)
⏰ Context from checks skipped due to timeout of 90000ms (3)
- GitHub Check: Redirect rules - masterpoint
- GitHub Check: Header rules - masterpoint
- GitHub Check: Pages changed - masterpoint
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (7)
content/blog/managing-google-workspace-with-terraform.md (7)
10-11: Close the<p>tag in the front-mattercalloutstring.The HTML in the
calloutfield opens a<p>element but never closes it, which can break page markup in some MD renderers.-callout: <p>👋 <b>If you're ready to take your infrastructure to the next level, we're here to help. We love to work together with engineering teams to help them build well-documented, scalable, automated IaC that make their jobs easier. <a href='/contact'>Get in touch!</a> +callout: <p>👋 <b>If you're ready to take your infrastructure to the next level, we're here to help.</b> We love to work together with engineering teams to help them build well-documented, scalable, automated IaC that make their jobs easier. <a href='/contact'>Get in touch!</a></p>
34-38: Minor grammar tweak – use plural “permissions”.
a permissionsounds odd with an uncountable noun. Small polish for readability.- Nobody remembers why or when a permission was changed. + Nobody remembers why or when permissions were changed.
40-41: Add a question mark after the parenthetical “dare we say”.The phrase is rhetorical, so punctuate it as a question.
- (and dare we say departments) become more distributed and harder to manage + (and dare we say departments?) become more distributed and harder to manage
44-46: Hyphenate compound adjective “open-source”.- a couple of open source solutions + a couple of open-source solutions
50-54: Tighten wording & fix “sign into” / “straight forward”.- 3. Easily manage SSO configurations allowing engineers to sign into multiple AWS accounts using their Google Identity. - 4. Intuitively organize user and team settings in a clear and straight forward approach. + 3. Easily manage SSO configurations, allowing engineers to sign in to multiple AWS accounts using their Google identity. + 4. Intuitively organize user and team settings in a clear and straightforward way.
63-64: Remove double punctuation.- files).. + files).
156-156: Fix ordered-list numbering to satisfy markdown-lint (MD029).Restarting at
1.avoids tooling warnings and keeps maintenance simple.-2. **Test validations in the variables.tf file.** +1. **Test validations in the variables.tf file.**
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
content/blog/managing-google-workspace-with-terraform.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
content/blog/managing-google-workspace-with-terraform.md
[grammar] ~35-~35: ‘a’ may be redundant when used with the uncountable noun ‘permission’.
Context: ...trategy. - Nobody remembers why or when a permission was changed. - Group membership becomes...
(A_UNCOUNTABLE_NOUN)
[typographical] ~40-~40: Except for inverted sentences, ‘dare we’ requires a question mark at the end of the sentence.
Context: ... manage across many different SaaS tools. It's important to have systems in place...
(MD_PRP_QUESTION_MARK)
[uncategorized] ~44-~44: If this is a compound adjective that modifies the following noun, use a hyphen.
Context: ...ith Terraform We looked at a couple of open source solutions and decided on using the Terr...
(EN_COMPOUND_ADJECTIVE_INTERNAL)
[style] ~45-~45: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...ervices, so this was an easy decision. We heard great things about [GAM](https://...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
[uncategorized] ~52-~52: Possible missing comma found.
Context: ...eam's permissions. 3. Easily manage SSO configurations allowing engineers to sign into multipl...
(AI_HYDRA_LEO_MISSING_COMMA)
[grammar] ~52-~52: The verb ‘sign into’ is not standard English, except in the context of the law (“The bill was signed into law”). Write “sign in to”. For websites and computers, other options are “log in to” or “log on to”.
Context: ...SO configurations allowing engineers to sign into multiple AWS accounts using their Googl...
(SIGN_INTO)
[uncategorized] ~53-~53: The adjective or adverb “straightforward” (= easy, precise) is spelled as one word.
Context: ...e user and team settings in a clear and straight forward approach. Here's the GitHub link for t...
(STRAIGHT_FORWARD_COMPOUND)
[typographical] ~63-~63: Two consecutive dots
Context: ...wn users.yaml and groups.yaml files).. 2. Create New Resources Start...
(DOUBLE_PUNCTUATION)
[uncategorized] ~110-~110: Possible missing comma found.
Context: ... Design Decisions to Make It Intuitive Below we point out a few Terraform module des...
(AI_HYDRA_LEO_MISSING_COMMA)
[uncategorized] ~291-~291: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short).
Context: ... care of the rest. We love this workflow and we hope it's useful to you and your tea...
(COMMA_COMPOUND_SENTENCE)
🪛 markdownlint-cli2 (0.17.2)
content/blog/managing-google-workspace-with-terraform.md
156-156: Ordered list item prefix
Expected: 1; Actual: 2; Style: 1/1/1
(MD029, ol-prefix)
🪛 GitHub Check: Trunk Check
content/blog/managing-google-workspace-with-terraform.md
[notice] 156-156: markdownlint(MD029)
[new] Ordered list item prefix
[failure] 1-1: prettier
Incorrect formatting, autoformat by running 'trunk fmt'
⏰ Context from checks skipped due to timeout of 90000ms (3)
- GitHub Check: Redirect rules - masterpoint
- GitHub Check: Header rules - masterpoint
- GitHub Check: Pages changed - masterpoint
🔇 Additional comments (1)
content/blog/managing-google-workspace-with-terraform.md (1)
1-296: Run repository formatter (trunk fmt/ Prettier) once wording changes are applied.Formatting is currently failing CI; running the project's auto-formatter will resolve any spacing and line-wrap issues introduced by edits.
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (5)
content/blog/managing-google-workspace-with-terraform.md (5)
35-36: Pluralise “permissions” and fix verb agreement-- Nobody remembers why or when a permission was changed. +- Nobody remembers why or when permissions were changed.
44-47: Hyphenate the compound adjective “open-source”-We looked at a couple of open source solutions +We looked at a couple of open-source solutions
50-54: Minor wording tweaks in the motivation list-3. Easily manage SSO configurations allowing engineers to sign into multiple AWS accounts using their Google Identity. +3. Easily manage SSO configurations, allowing engineers to sign in to multiple AWS accounts using their Google Identity. -4. Intuitively organize user and team settings in a clear and straight forward approach. +4. Intuitively organise user and team settings in a clear and straightforward manner.
62-64: Remove double punctuation-...create your own `users.yaml` and `groups.yaml` files).. +...create your own `users.yaml` and `groups.yaml` files).
156-156: Conform ordered-list numbering to project markdown-lint rule (1/1/1)
markdownlint(MD029) expects every ordered-list item to start with1..-2. **Test validations in the variables.tf file.** +1. **Test validations in the variables.tf file.**
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
content/blog/managing-google-workspace-with-terraform.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
content/blog/managing-google-workspace-with-terraform.md
[style] ~16-~16: To elevate your writing, try using an alternative expression here.
Context: ...wing Pains](#growing-pains) - [Why This Matters for Scaling Teams](#why-this-matters-fo...
(MATTERS_RELEVANT)
[style] ~30-~30: To elevate your writing, try using an alternative expression here.
Context: ...ious administrative tasks. ## Why This Matters for Scaling Teams In our small company...
(MATTERS_RELEVANT)
[grammar] ~35-~35: ‘a’ may be redundant when used with the uncountable noun ‘permission’.
Context: ...trategy. - Nobody remembers why or when a permission was changed. - Group membership becomes...
(A_UNCOUNTABLE_NOUN)
[typographical] ~40-~40: Except for inverted sentences, ‘dare we’ requires a question mark at the end of the sentence.
Context: ... manage across many different SaaS tools. It's important to have systems in place...
(MD_PRP_QUESTION_MARK)
[uncategorized] ~44-~44: If this is a compound adjective that modifies the following noun, use a hyphen.
Context: ...ith Terraform We looked at a couple of open source solutions and decided on using the Terr...
(EN_COMPOUND_ADJECTIVE_INTERNAL)
[style] ~45-~45: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...ervices, so this was an easy decision. We heard great things about [GAM](https://...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
[uncategorized] ~52-~52: Possible missing comma found.
Context: ...eam's permissions. 3. Easily manage SSO configurations allowing engineers to sign into multipl...
(AI_HYDRA_LEO_MISSING_COMMA)
[grammar] ~52-~52: The verb ‘sign into’ is not standard English, except in the context of the law (“The bill was signed into law”). Write “sign in to”. For websites and computers, other options are “log in to” or “log on to”.
Context: ...SO configurations allowing engineers to sign into multiple AWS accounts using their Googl...
(SIGN_INTO)
[uncategorized] ~53-~53: The adjective or adverb “straightforward” (= easy, precise) is spelled as one word.
Context: ...e user and team settings in a clear and straight forward approach. Here's the GitHub link for t...
(STRAIGHT_FORWARD_COMPOUND)
[typographical] ~63-~63: Two consecutive dots
Context: ...wn users.yaml and groups.yaml files).. 2. Create New Resources Start...
(DOUBLE_PUNCTUATION)
[uncategorized] ~110-~110: Possible missing comma found.
Context: ... Design Decisions to Make It Intuitive Below we point out a few Terraform module des...
(AI_HYDRA_LEO_MISSING_COMMA)
[uncategorized] ~291-~291: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short).
Context: ... care of the rest. We love this workflow and we hope it's useful to you and your tea...
(COMMA_COMPOUND_SENTENCE)
🪛 markdownlint-cli2 (0.17.2)
content/blog/managing-google-workspace-with-terraform.md
156-156: Ordered list item prefix
Expected: 1; Actual: 2; Style: 1/1/1
(MD029, ol-prefix)
🪛 GitHub Check: Trunk Check
content/blog/managing-google-workspace-with-terraform.md
[notice] 156-156: markdownlint(MD029)
[new] Ordered list item prefix
[failure] 1-1: prettier
Incorrect formatting, autoformat by running 'trunk fmt'
⏰ Context from checks skipped due to timeout of 90000ms (3)
- GitHub Check: Redirect rules - masterpoint
- GitHub Check: Header rules - masterpoint
- GitHub Check: Pages changed - masterpoint
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (1)
content/blog/managing-google-workspace-with-terraform.md (1)
10-10: Unclosed HTML tags incalloutbreak front-matter parsingThe
<p>and<b>tags aren’t closed, which can corrupt the YAML front-matter and break the Markdown renderer / site build. Close both tags (and prefer<strong>over<b>for semantic HTML).-callout: <p>👋 <b>If you're ready to take your infrastructure to the next level, we're here to help. We love to work together with engineering teams to help them build well-documented, scalable, automated IaC that make their jobs easier. <a href='/contact'>Get in touch!</a> +callout: <p>👋 <strong>If you're ready to take your infrastructure to the next level, we're here to help.</strong> We love to work together with engineering teams to help them build well-documented, scalable, automated IaC that make their jobs easier. <a href='/contact'>Get in touch!</a></p>
🧹 Nitpick comments (4)
content/blog/managing-google-workspace-with-terraform.md (4)
44-46: Hyphenate “open-source” when used adjectivally“As a compound adjective before a noun, add a hyphen.”
-We looked at a couple of open source solutions +We looked at a couple of open-source solutions
52-54: Minor grammar fixes in list item
- “sign into” → “sign in to” (standard usage)
- Comma after the introductory participial phrase improves readability.
-3. Easily manage SSO configurations allowing engineers to sign into multiple AWS accounts using their Google Identity. +3. Easily manage SSO configurations, allowing engineers to sign in to multiple AWS accounts using their Google Identity.
53-54: Spell “straightforward” as one word-4. Intuitively organize user and team settings in a clear and straight forward approach. +4. Intuitively organize user and team settings in a clear and straightforward approach.
62-64: Remove double period-... your own `users.yaml` and `groups.yaml` files).. +... your own `users.yaml` and `groups.yaml` files).
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
content/blog/managing-google-workspace-with-terraform.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
content/blog/managing-google-workspace-with-terraform.md
[style] ~16-~16: To elevate your writing, try using an alternative expression here.
Context: ...wing Pains](#growing-pains) - [Why This Matters for Scaling Teams](#why-this-matters-fo...
(MATTERS_RELEVANT)
[duplication] ~18-~18: Possible typo: you repeated a word.
Context: ...-terraform) - Getting Started With the Module - [Module Design Decisions](#module-design-decisi...
(ENGLISH_WORD_REPEAT_RULE)
[style] ~30-~30: To elevate your writing, try using an alternative expression here.
Context: ...ious administrative tasks. ## Why This Matters for Scaling Teams In our small company...
(MATTERS_RELEVANT)
[grammar] ~35-~35: ‘a’ may be redundant when used with the uncountable noun ‘permission’.
Context: ...trategy. - Nobody remembers why or when a permission was changed. - Group membership becomes...
(A_UNCOUNTABLE_NOUN)
[typographical] ~40-~40: Except for inverted sentences, ‘dare we’ requires a question mark at the end of the sentence.
Context: ... manage across many different SaaS tools. It's important to have systems in place...
(MD_PRP_QUESTION_MARK)
[uncategorized] ~44-~44: If this is a compound adjective that modifies the following noun, use a hyphen.
Context: ...ith Terraform We looked at a couple of open source solutions and decided on using the Terr...
(EN_COMPOUND_ADJECTIVE_INTERNAL)
[style] ~45-~45: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...ervices, so this was an easy decision. We heard great things about [GAM](https://...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
[uncategorized] ~52-~52: Possible missing comma found.
Context: ...eam's permissions. 3. Easily manage SSO configurations allowing engineers to sign into multipl...
(AI_HYDRA_LEO_MISSING_COMMA)
[grammar] ~52-~52: The verb ‘sign into’ is not standard English, except in the context of the law (“The bill was signed into law”). Write “sign in to”. For websites and computers, other options are “log in to” or “log on to”.
Context: ...SO configurations allowing engineers to sign into multiple AWS accounts using their Googl...
(SIGN_INTO)
[uncategorized] ~53-~53: The adjective or adverb “straightforward” (= easy, precise) is spelled as one word.
Context: ...e user and team settings in a clear and straight forward approach. Here's the GitHub link for t...
(STRAIGHT_FORWARD_COMPOUND)
[typographical] ~63-~63: Two consecutive dots
Context: ...wn users.yaml and groups.yaml files).. 2. Create New Resources Start...
(DOUBLE_PUNCTUATION)
[uncategorized] ~110-~110: Possible missing comma found.
Context: ...sers } ``` ## Module Design Decisions Below we point out a few Terraform module des...
(AI_HYDRA_LEO_MISSING_COMMA)
[uncategorized] ~291-~291: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short).
Context: ... care of the rest. We love this workflow and we hope it's useful to you and your tea...
(COMMA_COMPOUND_SENTENCE)
🪛 markdownlint-cli2 (0.17.2)
content/blog/managing-google-workspace-with-terraform.md
156-156: Ordered list item prefix
Expected: 1; Actual: 2; Style: 1/1/1
(MD029, ol-prefix)
🪛 GitHub Check: Trunk Check
content/blog/managing-google-workspace-with-terraform.md
[notice] 156-156: markdownlint(MD029)
[new] Ordered list item prefix
[failure] 1-1: prettier
Incorrect formatting, autoformat by running 'trunk fmt'
⏰ Context from checks skipped due to timeout of 90000ms (3)
- GitHub Check: Redirect rules - masterpoint
- GitHub Check: Header rules - masterpoint
- GitHub Check: Pages changed - masterpoint
🔇 Additional comments (1)
content/blog/managing-google-workspace-with-terraform.md (1)
115-121: Ordered-list numbering fails markdownlint (MD029)The second list item is numbered
2.but markdownlint enforces sequential1.prefixes. Prepend all ordered-list items with1.to satisfy linting or add<!-- markdownlint-disable MD029 -->before the list.
Co-authored-by: Dan Moore <[email protected]>
Co-authored-by: Dan Moore <[email protected]>
Co-authored-by: Dan Moore <[email protected]>
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (2)
content/blog/managing-google-workspace-with-terraform.md (2)
28-29: Remove the second “(CEO/CTO)” parentheticalYou already introduced Matt’s role earlier; repeating it here clutters the prose.
- Matt (CEO/CTO) had to remember + Matt had to remember
10-10: Front-mattercalloutstill has dangling HTML – must close<b>/<p>tagsThe same issue was raised in the previous review but remains unfixed; unclosed tags break Markdown rendering and can choke some YAML parsers.
-callout: <p>👋 <b>If you're ready to take your infrastructure to the next level, we're here to help. We love to work together with engineering teams to help them build well-documented, scalable, automated IaC that make their jobs easier. <a href='/contact'>Get in touch!</a> +callout: <p>👋 <strong>If you're ready to take your infrastructure to the next level, we're here to help.</strong> We love to work together with engineering teams to help them build well-documented, scalable, automated IaC that make their jobs easier. <a href='/contact'>Get in touch!</a></p>
🧹 Nitpick comments (4)
content/blog/managing-google-workspace-with-terraform.md (4)
32-38: Minor wording / grammar polish for bullet list
- “a permission” → permissions
- Capitalize first word consistently.
- Add Oxford comma for parallelism.
-- Nobody remembers why or when a permission was changed. +- Nobody remembers why or when permissions were changed. -- Onboarding/offboarding gets slower and error-prone. +- Onboarding/off-boarding gets slower and error-prone.
50-54: Fix “sign into” and “straight forward” typos-3. Easily manage SSO configurations allowing engineers to sign into multiple AWS accounts using their Google Identity. +3. Easily manage SSO configurations, allowing engineers to sign in to multiple AWS accounts using their Google identity. -4. Intuitively organize user and team settings in a clear and straight forward approach. +4. Intuitively organize user and team settings in a clear and straightforward approach.
63-64: Eliminate double period-`users.yaml` and `groups.yaml` files).. +`users.yaml` and `groups.yaml` files).
291-292: Comma splice – add a comma before “and”-We love this workflow and we hope it's useful to you and your team! +We love this workflow, and we hope it's useful to you and your team!
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
content/blog/managing-google-workspace-with-terraform.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
content/blog/managing-google-workspace-with-terraform.md
[style] ~16-~16: To elevate your writing, try using an alternative expression here.
Context: ...wing Pains](#growing-pains) - [Why This Matters for Scaling Teams](#why-this-matters-fo...
(MATTERS_RELEVANT)
[duplication] ~18-~18: Possible typo: you repeated a word.
Context: ...-terraform) - Getting Started With the Module - [Module Design Decisions](#module-design-decisi...
(ENGLISH_WORD_REPEAT_RULE)
[style] ~30-~30: To elevate your writing, try using an alternative expression here.
Context: ...ious administrative tasks. ## Why This Matters for Scaling Teams In our small company...
(MATTERS_RELEVANT)
[grammar] ~35-~35: ‘a’ may be redundant when used with the uncountable noun ‘permission’.
Context: ...trategy. - Nobody remembers why or when a permission was changed. - Group membership becomes...
(A_UNCOUNTABLE_NOUN)
[typographical] ~40-~40: Except for inverted sentences, ‘dare we’ requires a question mark at the end of the sentence.
Context: ... manage across many different SaaS tools. It's important to have systems in place...
(MD_PRP_QUESTION_MARK)
[uncategorized] ~44-~44: If this is a compound adjective that modifies the following noun, use a hyphen.
Context: ...ith Terraform We looked at a couple of open source solutions and decided on using the Terr...
(EN_COMPOUND_ADJECTIVE_INTERNAL)
[style] ~45-~45: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...ervices, so this was an easy decision. We heard great things about [GAM](https://...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
[uncategorized] ~52-~52: Possible missing comma found.
Context: ...eam's permissions. 3. Easily manage SSO configurations allowing engineers to sign into multipl...
(AI_HYDRA_LEO_MISSING_COMMA)
[grammar] ~52-~52: The verb ‘sign into’ is not standard English, except in the context of the law (“The bill was signed into law”). Write “sign in to”. For websites and computers, other options are “log in to” or “log on to”.
Context: ...SO configurations allowing engineers to sign into multiple AWS accounts using their Googl...
(SIGN_INTO)
[uncategorized] ~53-~53: The adjective or adverb “straightforward” (= easy, precise) is spelled as one word.
Context: ...e user and team settings in a clear and straight forward approach. Here's the GitHub link for t...
(STRAIGHT_FORWARD_COMPOUND)
[typographical] ~63-~63: Two consecutive dots
Context: ...wn users.yaml and groups.yaml files).. 2. Create New Resources Start...
(DOUBLE_PUNCTUATION)
[uncategorized] ~110-~110: Possible missing comma found.
Context: ...sers } ``` ## Module Design Decisions Below we point out a few Terraform module des...
(AI_HYDRA_LEO_MISSING_COMMA)
[uncategorized] ~291-~291: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short).
Context: ... care of the rest. We love this workflow and we hope it's useful to you and your tea...
(COMMA_COMPOUND_SENTENCE)
🪛 GitHub Check: Trunk Check
content/blog/managing-google-workspace-with-terraform.md
[notice] 156-156: markdownlint(MD029)
[new] Ordered list item prefix
[failure] 1-1: prettier
Incorrect formatting, autoformat by running 'trunk fmt'
🪛 markdownlint-cli2 (0.17.2)
content/blog/managing-google-workspace-with-terraform.md
156-156: Ordered list item prefix
Expected: 1; Actual: 2; Style: 1/1/1
(MD029, ol-prefix)
Co-authored-by: yangci <[email protected]>
Co-authored-by: yangci <[email protected]>
Co-authored-by: yangci <[email protected]>
Co-authored-by: yangci <[email protected]>
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (4)
content/blog/managing-google-workspace-with-terraform.md (4)
326-331: Same colon‐vs‐equals issue ingroupsvariable example- name: string, - email: string, - settings: optional(object({ - who_can_join: optional(string), + name = string + email = string + settings = optional(object({ + who_can_join = optional(string)
12-13: Unclosed<p>/<b>tag breaks front-matter parsingThe
calloutvalue is still missing both</b>and</p>, which can break the Markdown renderer and even crash some YAML parsers.-callout: <p>👋 <b>If you're ready to take your infrastructure to the next level, we're here to help. We love to work together with engineering teams to help them build well-documented, scalable, automated IaC that make their jobs easier. <a href='/contact'>Get in touch!</a> +callout: <p>👋 <strong>If you're ready to take your infrastructure to the next level, we're here to help.</strong> We love to work together with engineering teams to help them build well-documented, scalable, automated IaC that make their jobs easier. <a href='/contact'>Get in touch!</a></p>
335-338:merge()will panic onnullv.settingsIf a group omits
settings,v.settingsisnullandmerge()errors. Guard with an empty map.- for k, v in var.groups : k => merge(v.settings, { email = v.email }) + for k, v in var.groups : k => merge( + v.settings != null ? v.settings : {}, + { email = v.email } + )
248-255: Invalid HCL syntax inusersvariable exampleColons aren’t valid inside
object()type declarations; they must use=. This snippet will not parse if copied verbatim.- groups: optional(map(object({ - role: optional(string, "MEMBER"), - delivery_settings: optional(string, "ALL_MAIL"), - type: optional(string, "USER"), - })), {}), + groups = optional(map(object({ + role = optional(string, "MEMBER") + delivery_settings = optional(string, "ALL_MAIL") + type = optional(string, "USER") + })), {})
🧹 Nitpick comments (3)
content/blog/managing-google-workspace-with-terraform.md (3)
50-50: Typo: “manager” → “manage”…finding a better way to manager your Google Workspace accounts.
- One aspect of that is finding a better way to manager your Google Workspace accounts. + One aspect of that is finding a better way to manage your Google Workspace accounts.
60-63: Minor wording fixes (“sign in to” & “straightforward”)- allowing engineers to sign into multiple AWS accounts + allowing engineers to sign in to multiple AWS accounts - Organize `group` and `group_setting` variable inputs in clear and straight forward approach + Organize `group` and `group_setting` variable inputs in a clear and straightforward approach
161-161: Spelling: “goups” → “groups”
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
content/blog/managing-google-workspace-with-terraform.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
content/blog/managing-google-workspace-with-terraform.md
[style] ~18-~18: To elevate your writing, try using an alternative expression here.
Context: ...wing Pains](#growing-pains) - [Why This Matters for Scaling Teams](#why-this-matters-fo...
(MATTERS_RELEVANT)
[duplication] ~20-~20: Possible typo: you repeated a word.
Context: ...-terraform) - Getting Started With the Module - [Module Design Decisions](#module-design-decisi...
(ENGLISH_WORD_REPEAT_RULE)
[style] ~38-~38: To elevate your writing, try using an alternative expression here.
Context: ...ious administrative tasks. ## Why This Matters for Scaling Teams In our small company...
(MATTERS_RELEVANT)
[grammar] ~43-~43: ‘a’ may be redundant when used with the uncountable noun ‘permission’.
Context: ...trategy. - Nobody remembers why or when a permission was changed. - Group membership becomes...
(A_UNCOUNTABLE_NOUN)
[typographical] ~48-~48: Except for inverted sentences, ‘dare we’ requires a question mark at the end of the sentence.
Context: ...say departments) become more distributed. It is harder to manage user accounts ac...
(MD_PRP_QUESTION_MARK)
[uncategorized] ~54-~54: If this is a compound adjective that modifies the following noun, use a hyphen.
Context: ...ith Terraform We looked at a couple of open source solutions and decided on using the [Ter...
(EN_COMPOUND_ADJECTIVE_INTERNAL)
[style] ~55-~55: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...ervices, so this was an easy decision. We heard great things about [GAM](https://...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
[style] ~56-~56: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...'t need its full range of capabilities. We prefer to use declarative systems so we...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
[uncategorized] ~56-~56: Possible missing comma found.
Context: ...abilities. We prefer to use declarative systems so we know the user account and group s...
(AI_HYDRA_LEO_MISSING_COMMA)
[grammar] ~60-~60: The verb ‘sign into’ is not standard English, except in the context of the law (“The bill was signed into law”). Write “sign in to”. For websites and computers, other options are “log in to” or “log on to”.
Context: ...ers.yaml#L12-L18) allowing engineers to sign into multiple AWS accounts using their Googl...
(SIGN_INTO)
[uncategorized] ~61-~61: The adjective or adverb “straightforward” (= easy, precise) is spelled as one word.
Context: ...p_setting` variable inputs in clear and straight forward approach (more details on this in [Desi...
(STRAIGHT_FORWARD_COMPOUND)
[uncategorized] ~196-~196: Possible missing comma found.
Context: ...ttings ``` ## Module Design Decisions Below we point out a few Terraform module des...
(AI_HYDRA_LEO_MISSING_COMMA)
[uncategorized] ~381-~381: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short).
Context: ... care of the rest. We love this workflow and we hope it's useful to you and your tea...
(COMMA_COMPOUND_SENTENCE)
⏰ Context from checks skipped due to timeout of 90000ms (3)
- GitHub Check: Redirect rules - masterpoint
- GitHub Check: Header rules - masterpoint
- GitHub Check: Pages changed - masterpoint
|
|
||
| ## Managing Google Workspace with Terraform | ||
|
|
||
| We looked at a couple of open source solutions and decided on using the [Terraform Google Workspace provider](https://github.com/hashicorp/terraform-provider-googleworkspace). We often use SaaS-specific Terraform providers to [provision user accounts for DataDog](https://github.com/masterpointio/terraform-datadog-users) and other SaaS services, so this was an easy decision. |
There was a problem hiding this comment.
| We looked at a couple of open source solutions and decided on using the [Terraform Google Workspace provider](https://github.com/hashicorp/terraform-provider-googleworkspace). We often use SaaS-specific Terraform providers to [provision user accounts for DataDog](https://github.com/masterpointio/terraform-datadog-users) and other SaaS services, so this was an easy decision. | |
| We looked at a couple of open source solutions and decided on using the [Terraform Google Workspace provider](https://github.com/hashicorp/terraform-provider-googleworkspace). We often use SaaS-specific Terraform providers to [provision user accounts for DataDog](https://github.com/masterpointio/terraform-datadog-users) and other SaaS services, so this was an easy decision. |
"We often use SaaS-specific Terraform providers to provision user accounts for DataDog and other SaaS services, so this was an easy decision." hmm this sentence feels off?
Co-authored-by: yangci <[email protected]>
Co-authored-by: yangci <[email protected]>
Co-authored-by: yangci <[email protected]>
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (4)
content/blog/managing-google-workspace-with-terraform.md (4)
249-254: Invalid HCL syntax (:vs=) persistsColons inside
object()type declarations are still invalid HCL. This was flagged previously and must be corrected.
323-330: Same colon-vs-equals issue ingroupsvariable exampleThe object type block will not parse with colons; replace with equals signs.
335-338:merge(null, …)risk remains
v.settingscan benull; callingmerge()withnullwill error. Guard with a default empty map.
11-13: Close the dangling HTML tags incalloutfront-matterThe
<b>tag is never closed and the surrounding<p>tag is left open. This can break both YAML parsing and Hugo rendering.-callout: <p>👋 <b>If you're ready to take your infrastructure to the next level, we're here to help. We love to work together with engineering teams to help them build well-documented, scalable, automated IaC that make their jobs easier. <a href='/contact'>Get in touch!</a> +callout: <p>👋 <strong>If you're ready to take your infrastructure to the next level, we're here to help.</strong> We love to work together with engineering teams to help them build well-documented, scalable, automated IaC that make their jobs easier. <a href='/contact'>Get in touch!</a></p>
🧹 Nitpick comments (4)
content/blog/managing-google-workspace-with-terraform.md (4)
48-51: Typo – “manager” → “manage”- One aspect of that is finding a better way to manager your Google Workspace accounts. + One aspect of that is finding a better way to manage your Google Workspace accounts.
60-63: Minor wording and spelling fixes
- “sign into” → “sign in to” (preferred idiom)
- “straight forward” → “straightforward”
- allowing engineers to sign into multiple AWS accounts using their Google Identity. + allowing engineers to sign in to multiple AWS accounts using their Google Identity. - Organize `group` and `group_setting` variable inputs in clear and straight forward approach + Organize `group` and `group_setting` variable inputs in a clear and straightforward approach
120-123: Spelling – “practive” → “practice”- # version = "X.X.X" # it's a best practive to version pin modules + # version = "X.X.X" # it's a best practice to version-pin modules
161-163: Typo – “goups” → “groups”-And here's an example `groups.yaml` file, also leveraging YAML anchors to apply sane defaults to multiple goups. +And here's an example `groups.yaml` file, also leveraging YAML anchors to apply sane defaults to multiple groups.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
content/blog/managing-google-workspace-with-terraform.md(1 hunks)
🧰 Additional context used
🧠 Learnings (1)
content/blog/managing-google-workspace-with-terraform.md (1)
Learnt from: oycyc
PR: masterpointio/masterpoint.io#68
File: layouts/partials/head.html:0-0
Timestamp: 2025-07-02T01:48:08.985Z
Learning: In the masterpointio/masterpoint.io Hugo project, the user oycyc prefers to maintain consistency with existing code patterns rather than making isolated optimizations, even when technically better approaches are available. They prioritize consistency over over-optimization.
🪛 LanguageTool
content/blog/managing-google-workspace-with-terraform.md
[style] ~18-~18: To elevate your writing, try using an alternative expression here.
Context: ...wing Pains](#growing-pains) - [Why This Matters for Scaling Teams](#why-this-matters-fo...
(MATTERS_RELEVANT)
[duplication] ~20-~20: Possible typo: you repeated a word.
Context: ...-terraform) - Getting Started With the Module - [Module Design Decisions](#module-design-decisi...
(ENGLISH_WORD_REPEAT_RULE)
[style] ~38-~38: To elevate your writing, try using an alternative expression here.
Context: ...ious administrative tasks. ## Why This Matters for Scaling Teams In our small company...
(MATTERS_RELEVANT)
[grammar] ~43-~43: ‘a’ may be redundant when used with the uncountable noun ‘permission’.
Context: ...trategy. - Nobody remembers why or when a permission was changed. - Group membership becomes...
(A_UNCOUNTABLE_NOUN)
[typographical] ~48-~48: Except for inverted sentences, ‘dare we’ requires a question mark at the end of the sentence.
Context: ...say departments) become more distributed. It is harder to manage user accounts ac...
(MD_PRP_QUESTION_MARK)
[uncategorized] ~54-~54: If this is a compound adjective that modifies the following noun, use a hyphen.
Context: ...ith Terraform We looked at a couple of open source solutions and decided on using the [Ter...
(EN_COMPOUND_ADJECTIVE_INTERNAL)
[style] ~55-~55: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...ervices, so this was an easy decision. We heard great things about [GAM](https://...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
[style] ~56-~56: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...'t need its full range of capabilities. We prefer to use declarative systems so we...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
[uncategorized] ~56-~56: Possible missing comma found.
Context: ...abilities. We prefer to use declarative systems so we know the user account and group s...
(AI_HYDRA_LEO_MISSING_COMMA)
[grammar] ~60-~60: The verb ‘sign into’ is not standard English, except in the context of the law (“The bill was signed into law”). Write “sign in to”. For websites and computers, other options are “log in to” or “log on to”.
Context: ...ers.yaml#L12-L18) allowing engineers to sign into multiple AWS accounts using their Googl...
(SIGN_INTO)
[uncategorized] ~61-~61: The adjective or adverb “straightforward” (= easy, precise) is spelled as one word.
Context: ...p_setting` variable inputs in clear and straight forward approach (more details on this in [Desi...
(STRAIGHT_FORWARD_COMPOUND)
[uncategorized] ~196-~196: Possible missing comma found.
Context: ...ttings ``` ## Module Design Decisions Below we point out a few Terraform module des...
(AI_HYDRA_LEO_MISSING_COMMA)
[uncategorized] ~381-~381: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short).
Context: ... care of the rest. We love this workflow and we hope it's useful to you and your tea...
(COMMA_COMPOUND_SENTENCE)
⏰ Context from checks skipped due to timeout of 90000ms (3)
- GitHub Check: Redirect rules - masterpoint
- GitHub Check: Header rules - masterpoint
- GitHub Check: Pages changed - masterpoint
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (4)
content/blog/systematize-google-workspace-with-terraform.md (4)
47-50: Typo: “manager” → “manage”-One aspect of that is finding a better way to manager your Google Workspace accounts. +One aspect of that is finding a better way to manage your Google Workspace accounts.
59-60: Minor wording fixes (“sign into” / “straight forward”)-1. Easily manage [...] allowing engineers to sign into multiple AWS accounts using their Google Identity. +1. Easily manage [...] allowing engineers to sign in to multiple AWS accounts using their Google Identity. -2. Organize `group` and `group_setting` variable inputs in clear and straight forward approach [...] +2. Organize `group` and `group_setting` variable inputs in a clear and straightforward approach [...]
118-122: Typo: “practive” → “practice”-# version = "X.X.X" # it's a best practive to version pin modules +# version = "X.X.X" # it's a best practice to version-pin modules
158-161: Typo: “goups” → “groups”-And here's an example `groups.yaml` file, also leveraging YAML anchors to apply sane defaults to multiple goups. +And here's an example `groups.yaml` file, also leveraging YAML anchors to apply sane defaults to multiple groups.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
content/blog/systematize-google-workspace-with-terraform.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
content/blog/systematize-google-workspace-with-terraform.md
[style] ~17-~17: To elevate your writing, try using an alternative expression here.
Context: ...wing Pains](#growing-pains) - [Why This Matters for Scaling Teams](#why-this-matters-fo...
(MATTERS_RELEVANT)
[duplication] ~19-~19: Possible typo: you repeated a word.
Context: ...-terraform) - Getting Started With the Module - [Module Design Decisions](#module-design-decisi...
(ENGLISH_WORD_REPEAT_RULE)
[style] ~37-~37: To elevate your writing, try using an alternative expression here.
Context: ...ious administrative tasks. ## Why This Matters for Scaling Teams In our small company...
(MATTERS_RELEVANT)
[grammar] ~42-~42: ‘a’ may be redundant when used with the uncountable noun ‘permission’.
Context: ...trategy. - Nobody remembers why or when a permission was changed. - Group membership becomes...
(A_UNCOUNTABLE_NOUN)
[typographical] ~47-~47: Except for inverted sentences, ‘dare we’ requires a question mark at the end of the sentence.
Context: ...say departments) become more distributed. It is harder to manage user accounts ac...
(MD_PRP_QUESTION_MARK)
[uncategorized] ~53-~53: If this is a compound adjective that modifies the following noun, use a hyphen.
Context: ...ith Terraform We looked at a couple of open source solutions and decided on using the [Ter...
(EN_COMPOUND_ADJECTIVE_INTERNAL)
[style] ~54-~54: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...ervices, so this was an easy decision. We heard great things about [GAM](https://...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
[style] ~55-~55: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...'t need its full range of capabilities. We prefer to use declarative systems so we...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
[uncategorized] ~55-~55: Possible missing comma found.
Context: ...abilities. We prefer to use declarative systems so we know the user account and group s...
(AI_HYDRA_LEO_MISSING_COMMA)
[grammar] ~59-~59: The verb ‘sign into’ is not standard English, except in the context of the law (“The bill was signed into law”). Write “sign in to”. For websites and computers, other options are “log in to” or “log on to”.
Context: ...ers.yaml#L12-L18) allowing engineers to sign into multiple AWS accounts using their Googl...
(SIGN_INTO)
[uncategorized] ~60-~60: The adjective or adverb “straightforward” (= easy, precise) is spelled as one word.
Context: ...p_setting` variable inputs in clear and straight forward approach (more details on this in [Desi...
(STRAIGHT_FORWARD_COMPOUND)
[uncategorized] ~195-~195: Possible missing comma found.
Context: ...ttings ``` ## Module Design Decisions Below we point out a few Terraform module des...
(AI_HYDRA_LEO_MISSING_COMMA)
[uncategorized] ~380-~380: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short).
Context: ... care of the rest. We love this workflow and we hope it's useful to you and your tea...
(COMMA_COMPOUND_SENTENCE)
| --- | ||
| visible: true | ||
| draft: false | ||
| title: "Systematize Your Google Workspace Setup with Terraform" |
There was a problem hiding this comment.
I think this is a weak title.
Better options after directing AI to take me through a few rounds of exercises to write better titles,
- Simplify Google Workspace Automation with Masterpoint's Terraform Module
- A Scalable Way to Manage Google Workspace Users with Terraform
- Automate Google Workspace Onboarding with Terraform
- The DevOps Way to Manage Google Workspace Users at Scale
- The Platform Engineering Way to Manage Google Workspace Users
- Avoid Manual Google Workspace Admin Headaches with Terraform
There was a problem hiding this comment.
@westonplatter agreed, different title would be good. This tool was shared by the marketing strategist we're working with: https://coschedule.com/headline-analyzer
Want to try your favorites out with that tool and see what it comes up with?
There was a problem hiding this comment.
Did some testing in the headline analyzer and decided on,
The Platform Engineering Way to Manage Google Workspace Users
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (4)
content/blog/platform-engineering-way-to-manage-google-workspace-users.md (4)
49-49: Fix typo “manager” → “manage”.-One aspect of that is finding a better way to manager your Google Workspace accounts. +One aspect of that is finding a better way to manage your Google Workspace accounts.
120-120: Correct “best practive”.- # version = "X.X.X" # it's a best practive to version pin modules + # version = "X.X.X" # it's a best practice to version-pin modules
160-160: Fix misspelling “goups”.-And here's an example `groups.yaml` file, also leveraging YAML anchors to apply sane defaults to multiple goups. +And here's an example `groups.yaml` file, also leveraging YAML anchors to apply sane defaults to multiple groups.
53-56: Vary sentence openings to avoid repetitive “We …” pattern.The four consecutive sentences beginning with “We” read monotonously. Consider re-phrasing to improve flow, e.g.:
-We looked at a couple of open source solutions and decided on using the ... -We often use SaaS-specific Terraform providers to provision user accounts ... -We heard great things about GAM from a few colleagues, but we didn't need its full range of capabilities. -We prefer to use declarative systems so we know the user account and group settings we see in config files are indeed the values in production. +After evaluating several open-source options, the team chose the ... +Because SaaS-specific providers already power our DataDog user provisioning, the choice felt natural. +Colleagues praised GAM for its breadth, yet its imperative model exceeded our needs. +A declarative workflow lets the configuration files mirror production, keeping user and group settings transparent.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
content/blog/platform-engineering-way-to-manage-google-workspace-users.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
content/blog/platform-engineering-way-to-manage-google-workspace-users.md
[style] ~17-~17: To elevate your writing, try using an alternative expression here.
Context: ...wing Pains](#growing-pains) - [Why This Matters for Scaling Teams](#why-this-matters-fo...
(MATTERS_RELEVANT)
[style] ~37-~37: To elevate your writing, try using an alternative expression here.
Context: ...ious administrative tasks. ## Why This Matters for Scaling Teams In our small company...
(MATTERS_RELEVANT)
[style] ~54-~54: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...ervices, so this was an easy decision. We heard great things about [GAM](https://...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
[style] ~55-~55: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...'t need its full range of capabilities. We prefer to use declarative systems so we...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
[grammar] ~160-~160: Ensure spelling is correct
Context: ...hors to apply sane defaults to multiple goups. ```yaml --- _default_active_settings: ...
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (4)
content/blog/platform-engineering-way-to-manage-google-workspace-users.md (4)
49-49: Spelling: “manager” → “manage”.Minor typo breaks the sentence flow.
-One aspect of that is finding a better way to manager your Google Workspace accounts. +One aspect of that is finding a better way to manage your Google Workspace accounts.
53-53: Spelling: “mdoule” → “module”.-... we created a mdoule to [provision user accounts for DataDog] ... +... we created a module to [provision user accounts for DataDog] ...
120-120: Spelling: “practive” → “practice”.-# version = "X.X.X" # it's a best practive to version pin modules +# version = "X.X.X" # it's a best practice to version-pin modules
160-160: Spelling: “goups” → “groups”.-... apply sane defaults to multiple goups. +... apply sane defaults to multiple groups.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
content/blog/platform-engineering-way-to-manage-google-workspace-users.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
content/blog/platform-engineering-way-to-manage-google-workspace-users.md
[style] ~17-~17: To elevate your writing, try using an alternative expression here.
Context: ...wing Pains](#growing-pains) - [Why This Matters for Scaling Teams](#why-this-matters-fo...
(MATTERS_RELEVANT)
[style] ~37-~37: To elevate your writing, try using an alternative expression here.
Context: ...ious administrative tasks. ## Why This Matters for Scaling Teams In our small company...
(MATTERS_RELEVANT)
[grammar] ~53-~53: Ensure spelling is correct
Context: ...aaS products (for example, we created a mdoule to [provision user accounts for DataDog...
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
[style] ~54-~54: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...ogle users felt like an easy decision. We heard great things about [GAM](https://...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
[style] ~55-~55: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...'t need its full range of capabilities. We prefer to use declarative systems so we...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
[grammar] ~160-~160: Ensure spelling is correct
Context: ...hors to apply sane defaults to multiple goups. ```yaml --- _default_active_settings: ...
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (5)
content/blog/platform-engineering-way-to-manage-google-workspace-users.md (5)
49-49: Fix typo – “manager” ⇒ “manage”.-One aspect of that is finding a better way to manager your Google Workspace accounts. +One aspect of that is finding a better way to manage your Google Workspace accounts.
53-53: Correct misspelling of “module”.-... we created a mdoule to [provision user accounts for DataDog] ... +... we created a module to [provision user accounts for DataDog] ...
60-60: Use the single-word form “straightforward”.-... inputs in clear and straight forward approach ... +... inputs in a clear and straightforward approach ...
120-120: Typo in comment – “practive” ⇒ “practice”.- # version = "X.X.X" # it's a best practive to version pin modules + # version = "X.X.X" # it's a best practice to version-pin modules
160-160: Fix spelling of “groups”.-And here's an example `groups.yaml` file, also leveraging YAML anchors to apply sane defaults to multiple goups. +And here's an example `groups.yaml` file, also leveraging YAML anchors to apply sane defaults to multiple groups.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
content/blog/platform-engineering-way-to-manage-google-workspace-users.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
content/blog/platform-engineering-way-to-manage-google-workspace-users.md
[style] ~17-~17: To elevate your writing, try using an alternative expression here.
Context: ...wing Pains](#growing-pains) - [Why This Matters for Scaling Teams](#why-this-matters-fo...
(MATTERS_RELEVANT)
[style] ~37-~37: To elevate your writing, try using an alternative expression here.
Context: ...ious administrative tasks. ## Why This Matters for Scaling Teams In our small company...
(MATTERS_RELEVANT)
[grammar] ~53-~53: Ensure spelling is correct
Context: ...aaS products (for example, we created a mdoule to [provision user accounts for DataDog...
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
[style] ~54-~54: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...ogle users felt like an easy decision. We heard great things about [GAM](https://...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
[style] ~55-~55: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...'t need its full range of capabilities. We prefer to use declarative systems so we...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
[grammar] ~160-~160: Ensure spelling is correct
Context: ...hors to apply sane defaults to multiple goups. ```yaml --- _default_active_settings: ...
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
- GitHub Check: Redirect rules - masterpoint
- GitHub Check: Header rules - masterpoint
- GitHub Check: Pages changed - masterpoint
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
content/blog/platform-engineering-way-to-manage-google-workspace-users.md (1)
54-57: Minor style nit – repetitive sentence openings.Three consecutive sentences start with “We…”. Consider merging or re-phrasing for better flow.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
content/blog/platform-engineering-way-to-manage-google-workspace-users.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
content/blog/platform-engineering-way-to-manage-google-workspace-users.md
[style] ~17-~17: To elevate your writing, try using an alternative expression here.
Context: ...wing Pains](#growing-pains) - [Why This Matters for Scaling Teams](#why-this-matters-fo...
(MATTERS_RELEVANT)
[style] ~37-~37: To elevate your writing, try using an alternative expression here.
Context: ...ious administrative tasks. ## Why This Matters for Scaling Teams In our small company...
(MATTERS_RELEVANT)
[style] ~54-~54: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...ogle users felt like an easy decision. We heard great things about [GAM](https://...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
[style] ~55-~55: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...'t need its full range of capabilities. We prefer to use declarative systems so we...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
- GitHub Check: Redirect rules - masterpoint
- GitHub Check: Header rules - masterpoint
- GitHub Check: Pages changed - masterpoint
🔇 Additional comments (2)
content/blog/platform-engineering-way-to-manage-google-workspace-users.md (2)
10-10: Verify referenced image exists and is optimized.
/img/updates/managing-googleworkspace-with-terraform/preview-3.pngmust be present in the repo’s static assets and sized appropriately (≤ ~150 kB) to avoid 404s or slow loads.
16-23: Anchor slugs may not match generator rules.Many generators collapse consecutive dashes, so
#design-decision-1---implementing-integration-and-validation-testscould resolve to#design-decision-1-implementing-integration-and-validation-tests. Confirm and adjust the TOC links if needed.
| --- | ||
| visible: true | ||
| draft: false | ||
| title: "The Platform Engineering Way to Manage Google Workspace Users" | ||
| author: Weston Platter | ||
| date: 2025-07-17 | ||
| slug: platform-engineering-way-to-manage-google-workspace-users | ||
| description: "Migrate Google Workspace from ClickOps to Infrastructure as Code with our open source Terraform module. Includes design patterns and import examples." | ||
| tags: ["terraform", "google-workspace", "infrastructure-as-code", "automation"] | ||
| image: /img/updates/managing-googleworkspace-with-terraform/preview-3.png | ||
| callout: <p>👋 <b>If you're ready to take your infrastructure to the next level, we're here to help. We love to work together with engineering teams to help them build well-documented, scalable, automated IaC that make their jobs easier. <a href='/contact'>Get in touch!</a></p> | ||
| --- |
There was a problem hiding this comment.
Front-matter will fail YAML parsing – quote the callout value.
The callout field contains : and HTML tags but isn’t wrapped in quotes or a multi-line block, which will break most static-site builders (e.g. Hugo/Jekyll).
Patch:
-callout: <p>👋 <b>If you're ready to take your infrastructure to the next level, we're here to help. We love to work together with engineering teams to help them build well-documented, scalable, automated IaC that make their jobs easier. <a href='/contact'>Get in touch!</a></p>
+callout: |
+ <p>👋 <b>If you're ready to take your infrastructure to the next level, we're here to help. We love to work together with engineering teams to help them build well-documented, scalable, automated IaC that make their jobs easier. <a href="/contact">Get in touch!</a></p>📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| --- | |
| visible: true | |
| draft: false | |
| title: "The Platform Engineering Way to Manage Google Workspace Users" | |
| author: Weston Platter | |
| date: 2025-07-17 | |
| slug: platform-engineering-way-to-manage-google-workspace-users | |
| description: "Migrate Google Workspace from ClickOps to Infrastructure as Code with our open source Terraform module. Includes design patterns and import examples." | |
| tags: ["terraform", "google-workspace", "infrastructure-as-code", "automation"] | |
| image: /img/updates/managing-googleworkspace-with-terraform/preview-3.png | |
| callout: <p>👋 <b>If you're ready to take your infrastructure to the next level, we're here to help. We love to work together with engineering teams to help them build well-documented, scalable, automated IaC that make their jobs easier. <a href='/contact'>Get in touch!</a></p> | |
| --- | |
| --- | |
| visible: true | |
| draft: false | |
| title: "The Platform Engineering Way to Manage Google Workspace Users" | |
| author: Weston Platter | |
| date: 2025-07-17 | |
| slug: platform-engineering-way-to-manage-google-workspace-users | |
| description: "Migrate Google Workspace from ClickOps to Infrastructure as Code with our open source Terraform module. Includes design patterns and import examples." | |
| tags: ["terraform", "google-workspace", "infrastructure-as-code", "automation"] | |
| image: /img/updates/managing-googleworkspace-with-terraform/preview-3.png | |
| callout: | | |
| <p>👋 <b>If you're ready to take your infrastructure to the next level, we're here to help. We love to work together with engineering teams to help them build well-documented, scalable, automated IaC that make their jobs easier. <a href="/contact">Get in touch!</a></p> | |
| --- |
🤖 Prompt for AI Agents
In content/blog/platform-engineering-way-to-manage-google-workspace-users.md
lines 1 to 12, the callout field in the front-matter contains colons and HTML
tags but is not enclosed in quotes, causing YAML parsing errors. Fix this by
wrapping the entire callout value in double quotes or using a YAML multi-line
block style to ensure proper parsing by static-site generators.
what
Possible thumbnail images
Summary by CodeRabbit
Documentation
Chores
.gitignoreto exclude additional tool-specific files and directories.