Add optional reason to Requires and IgnoreIf

[l0rinc]

Fixes #1361

[codecov]

Codecov Report

Merging #1362 (7897558) into master (450dc1b) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #1362      +/-   ##
============================================
+ Coverage     79.20%   79.21%   +0.01%     
- Complexity     3951     3955       +4     
============================================
  Files           408      408              
  Lines         12299    12309      +10     
  Branches       1624     1627       +3     
============================================
+ Hits           9741     9751      +10     
  Misses         1975     1975              
  Partials        583      583              

Impacted Files	Coverage Δ
...k/runtime/extension/builtin/IgnoreIfExtension.java	100.00% <100.00%> (ø)
...k/runtime/extension/builtin/RequiresExtension.java	100.00% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 450dc1b...7897558. Read the comment docs.

[leonard84]

thanks @paplorinc

[kriegaex]

I know this was merged already, I just found it in the 2.1-M1 release notes. It seems that the reason parameter is undocumented in the Spock manual. Was that on purpose or does it deserve its own little issue?

[leonard84]

It is not directly on purpose. If you want to create a PR with some documentation you are welcome to do it.

Personally, I think it is discoverable enough, when I write code with an IDE I normally use auto complete, or show parameters to see what fields I have available.