Hi,
It would be good if module name in catnagios.info file starts with a capital letter.

CommentFileSizeAuthor
#7 afterpatchcatnagios-2.PNG7.12 KBAanal.addweb
#7 afterpatchcatnagios.PNG7.34 KBAanal.addweb
#7 before patch-Catnagios-2.PNG4.96 KBAanal.addweb
#7 before patch-Catnagios.PNG7.77 KBAanal.addweb
#6 2853119-module-name-should-be-capitalize-6.patch300 bytesswati.nuna
#3 2853119-module-name-should-be-capitalize.patch227 bytesswati.nuna

Comments

swati.nuna created an issue. See original summary.

swati.nuna’s picture

swati.nuna commented
Assigned: Unassigned » swati.nuna
swati.nuna’s picture

swati.nuna commented
StatusFileSize
new 2853119-module-name-should-be-capitalize.patch227 bytes

Here is updated patch file.

swati.nuna’s picture

swati.nuna commented
Status: Active » Needs review
swati.nuna’s picture

swati.nuna commented
swati.nuna’s picture

swati.nuna commented
StatusFileSize
new 2853119-module-name-should-be-capitalize-6.patch300 bytes

I have removed the newline at end of file.
Here is updated patch file.

Aanal.addweb’s picture

Aanal.addweb commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new before patch-Catnagios.PNG7.77 KB
new before patch-Catnagios-2.PNG4.96 KB
new afterpatchcatnagios.PNG7.34 KB
new afterpatchcatnagios-2.PNG7.12 KB

@swati.nuna, Thanks for both the patch corrections it works as i tested it by applying file in module & also over simplytest.me
PFA

swati.nuna’s picture

swati.nuna commented
Assigned: swati.nuna » Unassigned