Need to limit the lines exceeds than 80 characters standard error.

CommentFileSizeAuthor
#4 interdiff-3041361-3-4.txt11.22 KBWebbeh
#4 3041361-4.patch6.74 KBWebbeh
#3 3041361-3.patch2.9 KBakashkumar07
#2 pathauto-line-exceed-3041361.patch9.53 KBdaiwik.addweb

Comments

RoshniPatel.addweb created an issue. See original summary.

daiwik.addweb’s picture

daiwik.addweb commented
Status: Active » Needs review
StatusFileSize
new pathauto-line-exceed-3041361.patch9.53 KB

Here's the patch that resolves the error mentioned above. Kindly review it.

Thanks

akashkumar07’s picture

akashkumar07 commented
Category: Bug report » Task
StatusFileSize
new 3041361-3.patch2.9 KB

I hope, this patch should resolve the issue.

Webbeh’s picture

Webbeh
he/him/his
Location Georgia, USA
commented
Component: Code » Documentation
StatusFileSize
new 3041361-4.patch6.74 KB
new interdiff-3041361-3-4.txt11.22 KB

Per #3, can you please provide context and an interdiff in the future on how your patch differs from #2?

There are some trailing spaces throughout the document, plus the patch in #3 fails one hunk. Re-rolling patch with latest dev.

Additionally, rewrote Configuration to be more verbose on the steps and token implementations.

Patch and interdiff provided for review.

  • Berdir committed 3a4b6ad on 8.x-1.x authored by Webbeh 
    Issue #3041361 by Webbeh, RoshniPatel.addweb, AkashkumarOSL: README.txt...
berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Fixed

Thanks, committed.

Status: Fixed » Closed (fixed)

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