Skip to content

Fix typos and scope CI permissions - #216

Open
arijitroy003 wants to merge 1 commit into
dbt-labs:mainfrom
arijitroy003:fix/typos-and-ci-permissions
Open

Fix typos and scope CI permissions#216
arijitroy003 wants to merge 1 commit into
dbt-labs:mainfrom
arijitroy003:fix/typos-and-ci-permissions

Conversation

@arijitroy003

Copy link
Copy Markdown
Contributor

Summary

  • Fix "Terrraform" -> "Terraform" in README.md and docs/index.md
  • Fix "thie file" -> "this file" in README.md
  • Fix "CI/CI" -> "CI/CD" and "remoing" -> "removing" in main.py docstring
  • Replace overly broad permissions: write-all with scoped contents: read and pull-requests: write in .github/workflows/ci.yml

Test plan

  • Changes are cosmetic/config only — no behavior change
  • Verified patterns exist before fixing
  • CI permissions scoped to what the workflow actually uses (checkout, PR label editing, coverage comments)

- Fix "Terrraform" -> "Terraform" in README.md and docs/index.md
- Fix "thie file" -> "this file" in README.md
- Fix "CI/CI" -> "CI/CD" and "remoing" -> "removing" in main.py
- Replace overly broad "permissions: write-all" with scoped
  contents: read and pull-requests: write in CI workflow
@arijitroy003
arijitroy003 force-pushed the fix/typos-and-ci-permissions branch from a076c13 to acc6acf Compare July 16, 2026 20:53
@arijitroy003

Copy link
Copy Markdown
Contributor Author

Friendly bump — this has been open for ~12 days. Happy to address any feedback or make changes if needed. Let me know if there's anything I should adjust.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant