There is som errors in the composer.json file
Suggestions
Line 2:
"name": "drupal/view_user_email",
Line 5:
Type is set in line 3 so this line can be deleted.
Line 19:
Delete comma ,
The file should look like this.
{
"name": "drupal/view_user_email",
"type": "drupal-module",
"description": "Allow site administrator the ability to grant users of certain roles access to another users email address.",
"keywords": [
],
"homepage": "https://www.drupal.org/project/view_user_email",
"minimum-stability": "dev",
"authors": [
{
"name": "David Loor (davo20019)",
"homepage": "https://www.drupal.org/u/davo20019",
"role": "Maintainer"
}
],
"support": {
"issues": "https://www.drupal.org/project/issues/view_user_email",
"source": "http://cgit.drupalcode.org/view_user_email"
},
"require": {
"drupal/core": "^8.0.0 || ^9"
}
}
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | clean_patch - composer.png | 511.56 KB | neerajsingh |
| #2 | fixed_errors_in_composer.json-3174011.patch | 802 bytes | styrbaek |
Issue fork view_user_email-3174011
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #2
styrbaekI have created a patch for this issue.
My first patch so I am not sure if I am doing it right.
Comment #3
styrbaekComment #4
ant1The patch applies perfectly and seems to fix the warnings I'm receiving.
Setting to RTBC (as it is a simple fix)!
Comment #5
davo20019 commentedThanks styrbaek!
Comment #6
attheshow commentedThanks for the patch! Seems to apply cleanly.
Comment #7
neerajsinghThank you styrbaek..!

Patch is clean and works for me too. +1 to RTBC.
Comment #8
p4trizio commented+1 to RTBC
Comment #9
davo20019 commentedComment #12
tijsdeboeckI've added patch #2 to empty merge request !1. Can we please get this merged, as it's such a simple patch, and has been in use for over 2 years!
Comment #14
davo20019 commentedComment #15
davo20019 commentedComment #16
tijsdeboeckHi @davo20019, thanks for committing this! I noticed that @antoineh didn't receive a credit, while he was the first to review the patch.
(full disclosure: antoineh is my colleague, that's why I noticed this, but it seems fair, as neerajsingh also received a credit for testing)