search

upenn-cis1xx / camelot

A fully-modular OCaml style linter
https://opam.ocaml.org/packages/camelot/
Apache License 2.0
44 stars 5 forks source link

80 character line limit off by one #57

Closed Vighnesh-V closed 3 years ago

Vighnesh-V commented 3 years ago

Describe the bug The 80 character line limit check marks the wrong line in an off-by-one error.

To Reproduce Steps to reproduce the behavior:

  1. Open HW1 Box in codio
  2. Run linter.
  3. Both have off-by-one errors.

Expected behavior Probably an issue with the fact that Stream.from zero indexes lines, but editors use 1-indexing for line numbers.

Screenshots

image

image