Skip to content
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

Remove unsupp orted note for [allowed|forbidden]AccountIds #3712

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sayapeg
Copy link

@sayapeg sayapeg commented Jan 3, 2024

Description

Update the configuration options docs to reflect the support added in #508 for allowedAccountIds and forbiddenAccountIds.

Adding a new package?

If this pull request adds a new package:

  • The package's schema URL is correct.
  • The package metadata file, if present, contains:
    • a supported category (one of Cloud, Infrastructure, Network, Database, Monitoring, or Utility).
    • a description that explains what the package does.
    • a valid logo URL that points to a PNG whose dimensions conform to the others in this repo (e.g., 100x100).
    • a version number prefixed with v that corresponds with a valid GitHub release.
  • The package repo contains an Overview doc (/docs/_index.md) that includes:
    • a brief explanation of what the package is and what it does.
    • at least one representative example in all supported languages.
    • a front-matter property for the layout set to package.
  • The package repo contains an Installation and Configuration doc (/docs/installation-configuration.md) that includes:
    • links to SDKs in all supported languages.
    • a copyable command for installing the resource plugin if necessary.
    • an example of configuring the provider with pulumi config set.
    • an example of configuring the provider with environment variables.
  • Someone from the @pulumi/ai team has reviewed the PR.
  • Someone from the @pulumi/docs team has reviewed all documentation.

…Ids`

The support for these two parameters is implemented as part pulumi#508
@sean1588 sean1588 changed the title Remove unsupported note for [allowed|forbidden]AccountIds Remove unsupp orted note for [allowed|forbidden]AccountIds Jan 29, 2024
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.

None yet

1 participant