reek: docs/Syntax.md is missing

Description:

Usage of do/end blocks with ensure/rescue/else throws syntax error which points to non existent page.

[14]:Syntax: This file has unexpected token kENSURE
[https://github.com/troessner/reek/blob/master/docs/Syntax.md]

Environment:

  • ruby 2.5.0
  • reek 4.7.3

About this issue

  • Original URL
  • State: closed
  • Created 6 years ago
  • Reactions: 1
  • Comments: 19 (7 by maintainers)

Most upvoted comments

@krasnoukhov this should be fixed with Reek 5 which we’ll also push on CodeClimate.

I think this is not as much a missing docs file as a case that shouldn’t get to the point of reporting smells, but rather blow up with a syntax error.

(In this particular case we’re using a Ruby 2.4 parser to parse a Ruby 2.5 source, and that’s the culprit, but in general I wouldn’t consider syntax errors as smells.)