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

Add crowdstrike community package #5668

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

gtheodorio
Copy link

@gtheodorio gtheodorio commented Oct 10, 2024

add crowdstrike repo to package list

Description

Adding a new package?

If this pull request adds a new package:

  • The package's schema URL in this PR is correct.
  • The package metadata file, if present, contains:
    • a supported category (one of Cloud, Infrastructure, Network, Database, Monitoring, or Utility).
    • a valid plugin download URL. See Publish Your Package.
    • 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).
  • The package repo contains an Overview doc (/docs/_index.md) that includes:
    • [x ] 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.
  • The repository has:
    • a version tag prefixed with v that corresponds with a valid GitHub release and published package SDKs
  • A CODEOWNER has reviewed the PR.
  • A member of the @pulumi/docs team has reviewed all documentation.

add crowdstrike repo to package list
@ringods ringods changed the title Update package-list.json Add crowdstrike community package Oct 10, 2024
@iwahbe iwahbe removed the request for review from cnunciato November 6, 2024 09:15
@iwahbe iwahbe added the needs-triage Needs attention from the triage team label Nov 12, 2024
@mjeffryes
Copy link
Member

mjeffryes commented Nov 21, 2024

Thanks for the contribution @gtheodorio! And I apologize that no one replied on this PR right away.

We'd be happy to add this provider in the registry, but we do have a policy that new providers should have SDKs published in all Pulumi languages and that the examples on the docs pages support all Pulumi languages as well. It looks like this package is published everywhere except in Nuget for dotnet users. Would you be willing to publish a dotnet SDK and link to them in your docs?

@mjeffryes mjeffryes added awaiting-feedback Blocked on input from the author and removed needs-triage Needs attention from the triage team labels Nov 21, 2024
@gtheodorio
Copy link
Author

@mjeffryes Thanks for the update. I'll go ahead and create an SDK for dotnet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting-feedback Blocked on input from the author
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants