-
Notifications
You must be signed in to change notification settings - Fork 172
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
Update contributing documentation. Add new issue templates. #4327
base: 2.x
Are you sure you want to change the base?
Conversation
66b4ad4
to
959dd9a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello! A few nitpicking callouts things to change if you'd like, mostly preference things I'll leave up to your discretion. Otherwise looks great, thanks a bunch!
- type: textarea | ||
attributes: | ||
label: Description | ||
description: A concise description of the task or suggested implementation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nitpicking: I'm a stickler for using the word in the description/definition of something, so maybe: "A concise explanation of the task or suggested implementation."
- type: textarea | ||
attributes: | ||
label: Acceptance Criteria | ||
description: Tell us what should happen, how will we know when we're done. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nitpicking: "Tell us what should happen when the task is completed. How will we know when we're done?"
-------------------------------------- | ||
|
||
We recomend using DDEV with the DDEV-DKAN add-on as there are many helpful commands that will make it easier to get started. | ||
Once you have composer, docker, and DDEV `installed <https://dkan.readthedocs.io/en/latest/installation/index.html>`_, head over to `Getting Started with DDEV-DKAN <https://getdkan.github.io/ddev-dkan/getting-started.html>`_ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Period at the end of this for consistencies sake. "Getting started with DDEV-DKAN."
Coding Standards | ||
---------------- | ||
|
||
`Coder <http://drupal.org/project/coder>`_ is a tool to help write code for Drupal modules. It can detect and automatically fix coding standard errors. The project provides a coding standard for PHP_CodeSniffer based on the `Drupal coding standard <https://www.drupal.org/docs/develop/standards/php/php-coding-standards>`_ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Period at the end of here too: "Drupal coding standard."
Changes
QA Steps