vitest icon indicating copy to clipboard operation
vitest copied to clipboard

ci: add clean cache and pull request title validation workflow

Open dammy001 opened this issue 2 years ago • 3 comments

Description

This PR adds

  • clean cache workflow. https://docs.github.com/en/actions/using-workflows/caching-dependencies-to-speed-up-workflows#force-deleting-cache-entries.
  • Validate PR title workflow

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

  • [ ] It's really useful if your PR references an issue where it is discussed ahead of time. If the feature is substantial or introduces breaking changes without a discussion, PR might be closed.
  • [ ] Ideally, include a test that fails without this PR but passes with it.
  • [ ] Please, don't make changes to pnpm-lock.yaml unless you introduce a new test example.

Tests

  • [x] Run the tests with pnpm test:ci.

Documentation

  • [ ] If you introduce new functionality, document it. You can run documentation with pnpm run docs command.

Changesets

  • [x] Changes in changelog are generated from PR name. Please, make sure that it explains your changes in an understandable manner. Please, prefix changeset messages with feat:, fix:, perf:, docs:, or chore:.

dammy001 avatar Aug 03 '23 08:08 dammy001

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

Deploy Preview for fastidious-cascaron-4ded94 canceled.

Name Link
Latest commit 38d3b7341c487c544e6d6b323af1cc6b0b892437
Latest deploy log https://app.netlify.com/sites/fastidious-cascaron-4ded94/deploys/64d07fde7089540008ac6cf8

netlify[bot] avatar Aug 03 '23 08:08 netlify[bot]

I don't really understand the purpose of this PR, to be honest. What cache does it clear and don't we already do that? For PR titles, I don't think we should be too strict there since I can always rename any PR before merging.

sheremet-va avatar Jul 01 '24 16:07 sheremet-va

The team decided that this is not needed. The cache is already in place and we control the name of the PR when merging

sheremet-va avatar Nov 07 '24 14:11 sheremet-va