Potential fix for: Remove unnecessary linter warning from phpcs

[hjmsw]

Fixes #610

There would seem to be a couple of possible fixes:

phpcs.xml file included in this pull request includes the PSR2 ruleset from: https://github.com/squizlabs/PHP_CodeSniffer/blob/master/src/Standards/PSR2/ruleset.xml

The whitespace at the end of lines rule has been removed. Not sure whether to include this or not but with this rule in place there are a huge number of whitespace errors. Please see gist of phpcs output: https://gist.github.com/hjmsw/aa5e0608fa38acaeb9c5a6f6a1aa788d

And without whitespace at end of line rule, another gist:
https://gist.github.com/hjmsw/2ac4f13a0b92dad296369481d0cf872e

Another alternative would be to use the --standard=PSR2 command switch when running phpcs. The rules wanting to be ignored in #610 were ignored when I ran with the above command switch.

Checklist

• I have made a material change to the repo (functionality, testing, spelling, grammar)
• I have read the [Contribution Guide] and my PR follows them.
• I updated my branch with the master branch.

[thinkingserious]

Thanks @hjmsw! I've added this to our backlog for review.

[thinkingserious]

Hello @hjmsw,

Thanks again for the PR!

We appreciate your contribution and look forward to continued collaboration. Thanks!

Team SendGrid DX