Capitalize the name of the module, because module names are proper nouns.
Also according to Drupal standards, modules should include dependencies in the .info.yml file.
Dependencies should be namespaced in the format {project}:{module}, where {project} is the project name as it appears in the Drupal.org URL (e.g. drupal.org/project/views) and {module} is the module's machine name.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | owlcarousel-info-2994442-3.patch | 444 bytes | rudranil29 |
| #2 | owlcarousel-info-2994442-2.patch | 372 bytes | alonaoneill |
Comments
Comment #2
alonaoneill commentedI uploaded a patch that corrects the Capitalization and dependency namespacing in the .info.yml file.
Thanks!
Comment #3
dani3lr0se commentedThanks for the patch @alonaoneill. For some reason the patch failed to apply for me locally and the project isn't recognized by simplytest yet so it can't be tested there. Not sure why it didn't apply. It looks ok to me.
Comment #4
dani3lr0se commentedComment #5
srhcpa@debitsandcredits.com commentedHi Runranil,
When I ran this patch, I get the error: "No such file or directory".
I copied the patch to my public_html and ran this "git apply --ignore-space-change --ignore-whitespace owlcarousel-info-2994442-3.patch" from public_html.
Why do you think I am I getting this error?
Stephen
Hi Alona,
How do I run this patch? Can you give me a few instructions, please?
Stephen
Comment #6
rudranil29 commentedPlease apply this patch in the following format it will ignore whitespaces and space-changes in a file. In this way work for me
git apply --ignore-space-change --ignore-whitespace owlcarousel-info-2994442-3.patch
Comment #7
rudranil29 commentedComment #8
ipwa commentedThanks guys
Comment #9
ipwa commentedComment #11
ipwa commented