Skip to content

Update contribution process#29

Draft
richard-cox wants to merge 2 commits intorancher:mainfrom
richard-cox:process
Draft

Update contribution process#29
richard-cox wants to merge 2 commits intorancher:mainfrom
richard-cox:process

Conversation

@richard-cox
Copy link
Member

@richard-cox richard-cox commented Mar 6, 2025

Initial contribution process

@richard-cox richard-cox changed the title Update README.md Update contribution process Mar 6, 2025
@aalves08
Copy link
Member

aalves08 commented Apr 1, 2025

@richard-cox what do we need to move this forward? I see there are a couple of mentions to Troy but I don't remember exactly what we needed from him (at this moment I can't remember the context of all the mentions 😅 )

### What are the requirements for adding a project to this repository?

## Requirements
TODO: Confirm with Troy - verified partner
Copy link
Member Author

Choose a reason for hiding this comment

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

Do we need to run through the SUSE verified partner process for owners of extensions that we ship in this repo?

TODO: Confirm with Troy - verified partner

* Charts must be Helm 3 compatible.
TODO: Confirm with Troy - how supply and track business contact (if not included below)?
Copy link
Member Author

Choose a reason for hiding this comment

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

If we don't require them to become suse verified partners how to we track contact details?

* Compatibility annotations. For more information see [here](https://extensions.rancher.io/extensions/next/extensions-configuration#configurable-annotations)
* Be compatible with at least the current version of Rancher

TODO: require description, home url, maintainers in package.json
Copy link
Member Author

Choose a reason for hiding this comment

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

Ad specific requirements

- Impact on security, scaling and performance
- Code Review
- UX Review
- Documentation and test approach TODO: Confirm, do we include these?
Copy link
Member Author

Choose a reason for hiding this comment

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

To discuss, do we need to add more detail here, elsewhere or at all?

@richard-cox
Copy link
Member Author

@aalves08 added some PR comments that should help clarify the TODOs. Unfortunately, from my side, this won't get any TLC for a few months

@aalves08
Copy link
Member

aalves08 commented Apr 2, 2025

@aalves08 added some PR comments that should help clarify the TODOs. Unfortunately, from my side, this won't get any TLC for a few months

I know... I know... Don't worry. What you've written is already amazing 🙇

I'll try to move this forward in between my priority work.

Thanks! 🙏

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.

2 participants