Skip to content

Add additional plugin scaffolding rule#13

Open
PatAKnight wants to merge 1 commit into
mainfrom
update-some-docs
Open

Add additional plugin scaffolding rule#13
PatAKnight wants to merge 1 commit into
mainfrom
update-some-docs

Conversation

@PatAKnight

Copy link
Copy Markdown
Owner

Summary

Adds more rules to the scaffolding plugin cursor rule to ensure that we follow some specific steps when creating resources for plugins.

Ideally, we should allow for a round robin approach so that users are not stuck waiting for resources to come available before the next step. We should also include different resource options (plain deployment, operator, helm chart) for configuring integration points.

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • New plugin support (adds integration for a new RHDH plugin)
  • Documentation update
  • Refactoring (no functional changes)
  • CI/CD changes

Related Issues

Closes #(issue number)

Changes Made

  • Change 1
  • Change 2
  • Change 3

Checklist

General

  • I have read the CONTRIBUTING guide
  • My code follows the project's code conventions
  • I have tested my changes against a real cluster
  • If upgrading RHDH version, verified values.yaml is compatible with new chart

Security

  • No secrets, tokens, or credentials are included in this PR
  • Secret templates contain only placeholders (empty strings or <PLACEHOLDER>)
  • No hardcoded cluster URLs or API endpoints

For Plugin Contributions

  • Documentation added in docs/<plugin-name>.md
  • Configuration script added in scripts/config-<plugin-name>-plugin.sh
  • config-plugins.sh updated with new mappings
  • Secret templates updated with new placeholders
  • Resource manifests added (if required)
  • All new scripts are executable (chmod +x)

Testing Notes

Describe how you tested these changes:

Screenshots (if applicable)

Add screenshots to help explain your changes.

Signed-off-by: Patrick Knight <pknight@redhat.com>
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