wasmCloud icon indicating copy to clipboard operation
wasmCloud copied to clipboard

[FEATURE] Update Conventional Commit comment to include specific commit

Open lachieh opened this issue 1 year ago • 0 comments

Affected project(s)

  • [ ] documentation
  • [ ] examples
  • [ ] wasmCloud host
  • [ ] wasmCloud CLI (wash)
  • [ ] wasmCloud dashboard UI (washboard)
  • [ ] capability providers
  • [ ] provider bindgen
  • [ ] control interface client
  • [X] other / not sure

Is your feature request related to a problem? Please describe.

I couldn't easily tell from the bot comment which commit/s were incorrect

Describe the solution you'd like

Update the bot comment to include the errors for the specific commits.

Describe alternatives you've considered

No response

Additional context

The output from the lint check is below. There may be a way to get this info into a pipeline variable so that it can be used in the comment template.

Lint Error Ouput

2024/09/17 13:01:15 config file not found, using defaults
   • Starting pipeline: commit-pipeline
   • Starting analysis of commits on branch HEAD
   • Commit found: [hash] e[5](https://github.com/wasmCloud/wasmCloud/actions/runs/10903550828/job/30257880599#step:4:6)a4ed4a202809bdcdac218b345ffb9b3c65b00a [message] Merge 9b9[6](https://github.com/wasmCloud/wasmCloud/actions/runs/10903550828/job/30257880599#step:4:7)69a793469e6323f774587d02c70a35a060da into ab7712b6504fb30[7](https://github.com/wasmCloud/wasmCloud/actions/runs/10903550828/job/30257880599#step:4:8)fd056daf9a861e0ec8d5c39f

   • Commit found: [hash] 9b9669a793469e6323f774587d02c70a35a060da [message] refactor!(lattice-client-core): move retry logic to client, update client options to require specific connection

lattice client now requires specific connection creator function

Signed-off-by: Lachlan Heywood <[email protected]>

   • Commit found: [hash] 97ae3e98c04bae0fcea43da3bc59823c966ae347 [message] correct build output for commonjs projects
   • Commit found: [hash] 6f00681e14365720de9f108e9e13be8ab87a452d [message] update vite-plugin-dts after fixing qmhc/vite-plugin-dts#376
   • Commit found: [hash] 3d5586d403783f0a7cf9f9357077d9d47525a5b3 [message] add 'initial' state to lattice connection to better handle connection. use string unions instead of enum for better typed experience.
   • Commit found: [hash] a1016345d98603455d21e93e8dd188fe0d2978b4 [message] update to vite for build and clean up tasks/deps
   • 1 commits filtered out   
   • Found 5 commit to check  
   • 2 of 6 commits are not conventional commit compliant
   • Expected format is for example:      chore(ci): this is a test
   • Please see https://www.conventionalcommits.org for help on how to structure commits
   • All pipelines complete   
   • [commit-pipeline] Not all commits are conventional commits, please check the commits listed above
   ⨯ pipeline: commit-pipeline, error: Not all commits are conventional commits, please check the commits listed above
+------------------------------------------+------------------+------------------------------------------------------------------------------------------------------------------+
| HASH                                     | FAILURE          | TEXT                                                                                                             |
+------------------------------------------+------------------+------------------------------------------------------------------------------------------------------------------+
| 9b9669a793469e6323f774587d02c70a35a060da | category missing | refactor!(lattice-client-core): move retry logic to client, update client options to require specific connection |
|                                          |                  |                                                                                                                  |
|                                          |                  | lattice client now requires specific connection creator function                                                 |
|                                          |                  |                                                                                                                  |
|                                          |                  | Signed-off-by: Lachlan Heywood <[email protected]>                                                |
|                                          |                  |                                                                                                                  |
| a1016345d98603455d21e93e8dd188fe0d2978b4 | malformed scope  | update to vite for build and clean up tasks/deps                                                                 |
+------------------------------------------+------------------+------------------------------------------------------------------------------------------------------------------+
some errors were found
Error: Process completed with exit code 1.

lachieh avatar Sep 17 '24 13:09 lachieh