practices icon indicating copy to clipboard operation
practices copied to clipboard

Document standard CI tests

Open nottrobin opened this issue 8 years ago • 6 comments

Thing we should test in all projects:

  • sass linting
  • flake8
  • pa11y
  • percy maybe?
  • link checker
  • HTML validation

And then any unit tests, of course. But that's just nice to have

nottrobin avatar Sep 11 '17 18:09 nottrobin

I was wondering if this CI tests could/should be added to generator-canonical-webteam?

So, it asks if you would like to update the Travis file. If so, detects and updates missing CI dependences and drops a Travis file containing this list of common CI tasks to the project.

Then we can maintain a common CI across all of our projects a little more easily.

anthonydillon avatar Sep 12 '17 07:09 anthonydillon

Absolutely it should. Travis already takes care of setting up flake8 and sass linting. But it doesn't mean it doesn't need to be documented here.

Although we hope the generator is used to set up new projects, there are cases where it won't work, or someone just doesn't know they're supposed to use it.

Our practices should be defined separately and in addition to the tools that help enforce them.

nottrobin avatar Sep 12 '17 07:09 nottrobin

The list looks good and I'd also question percy. It regularly 'fails' and requires manual intervention when small styling changes are made that should be picked up by a design -/+ 1.

Lukewh avatar Oct 09 '17 08:10 Lukewh

@Lukewh Added discussion about Percy to the dev catchup agenda.

nottrobin avatar Oct 09 '17 14:10 nottrobin

In dev catch-up ^ we decided that Percy should indeed be included as a standard CI test.

nottrobin avatar Oct 19 '17 14:10 nottrobin

^ As long as we disable dynamic content.

Lukewh avatar Oct 19 '17 14:10 Lukewh