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

(DOC) add PR template #16

Closed
wants to merge 1 commit into from
Closed

(DOC) add PR template #16

wants to merge 1 commit into from

Conversation

bmarwell
Copy link

No description provided.

@rmannibucau
Copy link
Contributor

Hi @bmarwell ,

I know it is a common ASF template but I personally hate this verbose checklist increasing a lot the contribution entry cost so I would minimize it as much as possible.
It is also not relevant to run mvn verify for meecrowave.

So I'm tempted, if we want a template, to write a real template and add a link to a doc (website) page for the legal/(i)CLA details.
Maybe something around:

# Issue

<write some description of the issue>

# Solution

<how I solve the issue by this PR>

TIP: if possible link the issue to a jira ticket and ensure the feature/bugfix is tested.

wdyt?

@bmarwell bmarwell closed this by deleting the head repository Apr 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants