Skip to content

Conversation

lancergr
Copy link
Contributor

No description provided.

@lancergr lancergr self-assigned this Oct 20, 2017
@coveralls
Copy link

coveralls commented Oct 20, 2017

Coverage Status

Coverage remained the same at 32.082% when pulling 78150bb on github-templates into b4f5f0d on develop.

@@ -0,0 +1,10 @@
| Q | A
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's do Question and answers rather then Q and A.

@@ -0,0 +1,5 @@
| Q | A
Copy link
Contributor

Choose a reason for hiding this comment

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

same here

| ---------------- | ---
| Request type | bugfix/feature
| Tested scenarios | Description of tested scenarios
| Fixed issues | #-prefixed issue number(s), if any
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe better to show these questions like this:

Description

<! Provide a description of the changes proposed in the pull request ->

Fixed Issues (if relevant)

  1. #<issue_number>: Issue title
  2. ...

Tested scenarios

  1. ...
  2. ...

Copy link
Contributor

@rikterbeek rikterbeek Oct 20, 2017

Choose a reason for hiding this comment

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

example see: magento/magento2#11576

@coveralls
Copy link

coveralls commented Oct 20, 2017

Coverage Status

Coverage remained the same at 32.082% when pulling 45999ff on github-templates into b4f5f0d on develop.

@coveralls
Copy link

coveralls commented Oct 26, 2017

Coverage Status

Coverage remained the same at 32.082% when pulling 3f0fe07 on github-templates into b4f5f0d on develop.

@lancergr lancergr merged commit 3547f4e into develop Oct 27, 2017
@lancergr lancergr deleted the github-templates branch October 27, 2017 09:59
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.

5 participants