anza-xyz / move

Move compiler targeting llvm supported backends
https://discord.gg/wFgfjG9J
Apache License 2.0
108 stars 33 forks source link

ci: extract check logic to an individual job #331

Closed yihau closed 1 year ago

yihau commented 1 year ago

Problem

https://github.com/solana-labs/move/pull/327#issuecomment-1705412044

the action is too active to parse those comments instead of commands

Summary of Changes

add an extract job, check, before entrypoint. it is used to check format and auth

demo pr: https://github.com/yihau/solana-move/pull/30

there are 4 comments in the PR. it means there are 4 corresponding actions.

  1. https://github.com/yihau/solana-move/actions/runs/6082440542 (skip due to format)
  2. https://github.com/yihau/solana-move/actions/runs/6082442835 (skip due to auth)
  3. https://github.com/yihau/solana-move/actions/runs/6082443717 (skip due to format)
  4. https://github.com/yihau/solana-move/actions/runs/6082444366 (success)

the action only run when 1) the command is using command format and 2) it is created from a qualified user

demo issue: https://github.com/yihau/solana-move/issues/31

https://github.com/yihau/solana-move/actions/runs/6082472902 (skip due to it is an issue comment)