PR Template Requests Running Tests But Gives No Instruction

Update: Resolved. There is a link—outside the PR description—to the contribution guide. Also, #17713.

---

Is your feature request related to a problem? Please describe Yes. My PR is #17702, but anyone opening a PR for a UI change may be requested to run Jest or Chromatic tests.

There is no instruction for testing, and the process was too much for so small a change, yet the request was still made.

Describe the solution you’d like A link in the Pull Request template to instructions for how to perform tests requested.

I support requesting testing, but I do not support the unexpected trouble when trying to test. See “Additional context”.

Describe alternatives you’ve considered

1. Researching on my own how to run the requested tests. See “Additional context”.
2. Improving the wording of the PR template to disregard testing for a minuscule changes.

Are you able to assist to bring the feature to reality? Yes, but I am too angry at the moment to promise to assist beyond creating this ticket.

Additional context

I took many steps to run the test suite, but eventually failed. _{Click for details.}

1. Clone my fork of the repo.

2. Run yarn install.

3. Run yarn jest.

4. See tests fail because dev dependencies were not installed.

5. Fail to install all dependencies because I was using a command for Yarn Classic.

6. Determine my yarn version… version 3.

7. Fail to learn how to install all dev dependencies with Yarn 3.

8. Learn how to downgrade to Yarn Classic.

9. Install all dev dependencies.

10. Be asked for which version to use for every @storybook/... dev dependency! [^1]

    Couldn't find any versions for "@storybook/svelte" that matches "workspace:*"
    ? Please choose a version of "@storybook/svelte" from this list: 
    

    I chose 6.5.0-alpha.47 for every options, because I am impatient and annoyed.

11. Be lambasted with many warnings.

12. Receive one error: error An unexpected error occurred: "expected workspace package to exist for \"react-router-dom\"".

13. Run yarn jest.

14. Receive more reports of missing dependencies. See snippet.

15. Give up!

[^1]: Why is a known working version of each dependency not pinned?!

[shilman]

Jeepers creepers!! I just released https://github.com/storybookjs/storybook/releases/tag/v6.5.0-alpha.52 containing PR #17713 that references this issue. Upgrade today to the @next NPM tag to try it out!

npx sb upgrade --prerelease

[yannbf]

No worries @tacc-wbomar! I totally understand your frustration and I hope others don’t go through similar experience.

I see value in improving the PR template and I’m looking forward to seeing yet another contribution from you!