Attached patch fixes coding standard issues.

CommentFileSizeAuthor
#3 Before Patch.png155.2 KBriddhi.addweb
#3 After patch.png56.05 KBriddhi.addweb
#2 interdiff.txt511 bytesMunavijayalakshmi
#2 fix_coding_standard_nodeblock-2849999-2.patch2.88 KBMunavijayalakshmi
fix_coding_standard_nodeblock_0.patch2.69 KBSubhransu.addweb

Comments

denisha.addweb created an issue. See original summary.

Munavijayalakshmi’s picture

Munavijayalakshmi commented
StatusFileSize
new fix_coding_standard_nodeblock-2849999-2.patch2.88 KB
new interdiff.txt511 bytes 
fix_coding_standard_nodeblock_0.patch:53: trailing whitespace.
      } 
Checking patch nodeblock.install...
Applied patch nodeblock.install cleanly.
warning: 1 line adds whitespace errors.

Fixed the whitespace error, @file block missing and attached new patch.

riddhi.addweb’s picture

riddhi.addweb commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new After patch.png56.05 KB
new Before Patch.png155.2 KB

@Munavijayalakshmi,Thanks for the patch it works well , i test this patch in simplytest.Me using coder module & it removes all errors of coding standard.PFA result of after & before patch.

Johnny vd Laar’s picture

Johnny vd Laar commented
Status: Reviewed & tested by the community » Fixed

Thanks for the patch.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.