xen-orchestra icon indicating copy to clipboard operation
xen-orchestra copied to clipboard

feat: run all tests in CI

Open julien-f opened this issue 2 years ago • 3 comments

Check list

Check if done, if not relevant leave unchecked.

  • [ ] PR reference the relevant issue (e.g. Fixes #007 or See xoa-support#42)
  • [ ] if UI changes, a screenshot has been added to the PR
  • [ ] documentation updated
  • CHANGELOG.unreleased.md:
    • [ ] enhancement/bug fix entry added
    • [ ] list of packages to release updated (${name} v${new version})
  • I have tested added/updated features (and impacted code)
    • [ ] unit tests (e.g. cron/parse.spec.js)
    • [ ] if xo-server API changes, the corresponding test has been added to/updated on xo-server-test
    • [ ] at least manual testing

Process

  1. create a PR as soon as possible
  2. mark it as WiP: (Work in Progress) if not ready to be merged
  3. when you want a review, add a reviewer (and only one)
  4. if necessary, update your PR, and re- add a reviewer

From the Four Agreements:

  1. Be impeccable with your word.
  2. Don't take anything personally.
  3. Don't make assumptions.
  4. Always do your best.

julien-f avatar Nov 24 '22 15:11 julien-f

@fbeauchamp Some tests are failing, I believe they are related to code you worked on if you want to take a look.

julien-f avatar Nov 24 '22 16:11 julien-f

@fbeauchamp Some tests are failing, I believe they are related to code you worked on if you want to take a look.

I am waiting for the check to end and will fix them . Thanks you

fbeauchamp avatar Nov 24 '22 16:11 fbeauchamp

Thank you! Let me know if you need help or if there are errors you want me to handle :slightly_smiling_face:

julien-f avatar Nov 24 '22 16:11 julien-f