3634132: coding standards

Phil Nortonrequested to merge
3634132-coding-standards-fixes into 1.0.x
  • 3634132: Removed the raw tag from the twig template. Even though this is used in a non-html setting, it's still not a great idea to leave raw in template files.
  • 3634132: Updated the .gitlab-ci.yml file to bring it into line with the template from Drupal.
  • 3634132: Coding standards fixes for the .module file.
  • 3634132: Added more description to the permission of the module.
  • 3634132: Added the node view permission to the route access to the RSS feed.
  • 3634132: Coding standards fix in the NodeRssHooks class.
  • 3634132: Updated the NodeRssController class to fix coding standards and make the permission more precise.
  • 3634132: Coding standards fixes for the NodeRssPathProcessor class. Also corrected some the regular expressions in use to only allow \rss to be used.
  • 3634132: Added more unit tests for the module to capture different permissions and routes. Fixed some coding standards things and a typo.

Closes #3634132

Merge request reports