Skip to content

Add Skill Localization test for Handlebars

Ryan Walder requested to merge ryanw/ci-skill-localization into master

Intent

We often commit code which applies localization to Skills incorrectly (See !1113 (merged)).

This adds a CI test to try and safeguard against that.


  • This is meant for a hotfix
  • This is meant for the next release (see milestone)
  • This needs more reviewers than normal; there may be controversy or high complexity
  • This intentionally introduces regressions that will be addressed later
  • There is/will be documentation changes on the wiki
  • Please do not send commits here without coordinating closely with the owner
  • This is a Build System change

Related Issues

  • N/A

Steps to Test

  1. Checkout this branch
  2. Add {{localize skill.name}} somewhere in a hbs file
  3. Run CI Locally / Run the bash script manually
  4. Test fails and tell you how to fix it

Additional Notes

CI will fail until !1113 (merged) is merged into !1109 (merged) which is then merged into dev

Edited by Ryan Walder

Merge request reports