Uploaded image for project: 'PHP Driver: Library'
  1. PHP Driver: Library
  2. PHPLIB-1138

Increase line length soft limit to 120 characters

    • Type: Icon: Task Task
    • Resolution: Works as Designed
    • Priority: Icon: Minor - P4 Minor - P4
    • None
    • Affects Version/s: None
    • Component/s: None

      PHPLIB currently does not enforce a line limit for code, but most comments wrap at 80 characters. In this thread from mongodb/mongo-php-library#1075, jerome.tamarelle@mongodb.com suggested using 120, which is the current default in PhpStorm. I don't believe the current limit is documented anywhere, so we should consider doing that as well with the switch.

      If we end up enforcing this in the PHPCS configuration, we should also check if there's a way to allow exceptions for URLs (see: squizlabs/PHP_CodeSniffer#766).

      Additionally, we can look into re-wrapping existing comments as part of this task.

            Assignee:
            jerome.tamarelle@mongodb.com Jérôme Tamarelle
            Reporter:
            jmikola@mongodb.com Jeremy Mikola
            Votes:
            1 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: