Skip to content

Conversation

@tlecomte
Copy link

@tlecomte tlecomte commented Nov 4, 2025

Changes

Reword the following documentation pages:

  • databricks_grant: it looks like the introduction text was copied from databricks_grants (plural), so reword to make it clearer that databricks_grant applies to a single securable+principal combination
  • databricks_grant and databricks_grants: a minor reword on users and groups are referred to as principals

Tests

Look at the markdown renders.

@tlecomte tlecomte requested review from a team as code owners November 4, 2025 09:26
@tlecomte tlecomte requested review from parthban-db and removed request for a team November 4, 2025 09:26
@github-actions
Copy link

github-actions bot commented Nov 4, 2025

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/terraform

Inputs:

  • PR number: 5182
  • Commit SHA: 2012bde856b1044586b60901fc391eb97f663415

Checks will be approved automatically on success.

@alexott alexott requested a review from Copilot November 7, 2025 14:59
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR improves the clarity of documentation for Unity Catalog grant resources by clarifying the scope of databricks_grant and standardizing terminology around principals.

  • Rewording the introductory text for databricks_grant to clarify it manages grants for a single securable+principal combination
  • Standardizing the reference to users and groups as "principals" across both grant documentation pages
  • Adding changelog entry for the documentation improvements

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
docs/resources/grant.md Clarified that databricks_grant is authoritative for a specific securable+principal pair, and standardized principal terminology
docs/resources/grants.md Standardized the phrasing around principals from "called principals in general" to "referred to as principals"
NEXT_CHANGELOG.md Added changelog entry documenting the documentation improvements

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@alexott alexott added the docs add explicit documentation about corner case label Nov 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs add explicit documentation about corner case

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants