ci(label): skip duplicated build on PR creation

[clearloop]

Resolves #3148
Resolves #3142

• ( after label checks ) process build only if the previous build is skipped
• mark the name of the triggered builds
• more details of the shadow build
• workaorund for Replace log::error! with log::trace! #3154
• introduce timeout for checks

Tests

show case of skipping duplicated build: https://github.com/clearloop/gear/pull/2
show case of named builds: https://github.com/gear-tech/gear/actions/runs/5943456240/job/16118640453

@gear-tech/dev

[shamilsan]

Need to add A4-insubstantial to CI triggers too.

[clearloop]

@CodiumAI-Agent /review

[CodiumAI-Agent]

PR Analysis

• 🎯 Main theme: Improving the CI/CD pipeline by skipping duplicated builds and introducing timeouts for checks.
• 📌 Type of PR: Enhancement
• ✨ Focused PR: Yes, all changes are related to improving the CI/CD pipeline.
• 🔒 Security concerns: No security concerns found

PR Feedback

• General suggestions: The PR seems to be well-structured and focused on improving the CI/CD pipeline. However, it would be beneficial to add tests to ensure the new changes work as expected. Also, consider adding error handling for unexpected cases to make the code more robust.

• 🤖 Code feedback:

  • relevant file: .github/actions/label/build.js
    suggestion: Consider adding error handling for the asynchronous operations. This can help to catch and handle any potential errors that might occur during the execution of these operations. [important]
    relevant line: "const { data: { check_runs }, } = await github.rest.checks.listForRef({ owner, repo, ref: REF, });"

  • relevant file: .github/actions/label/build.js
    suggestion: It would be beneficial to add comments explaining the logic behind the 'skip' function. This can help other developers understand the code more easily. [medium]
    relevant line: "const skip = async ({ core, github }) => {"

  • relevant file: .github/actions/label/build.js
    suggestion: Consider using more descriptive variable names. For example, 'linux' could be renamed to something like 'isLinuxLabel'. This can make the code more readable. [medium]
    relevant line: "const linux = LABEL === 'A0-pleasereview' || LABEL === 'A4-insubstantial' || LABEL === 'A2-mergeoncegreen';"

How to use

Tag me in a comment '@CodiumAI-Agent' and add one of the following commands:
/review [-i]: Request a review of your Pull Request. For an incremental review, which only considers changes since the last review, include the '-i' option.
/describe: Modify the PR title and description based on the contents of the PR.
/improve [--extended]: Suggest improvements to the code in the PR. Extended mode employs several calls, and provides a more thorough feedback.
/ask <QUESTION>: Pose a question about the PR.
/update_changelog: Update the changelog based on the PR's contents.

To edit any configuration parameter from configuration.toml, add --config_path=new_value
For example: /review --pr_reviewer.extra_instructions="focus on the file: ..."
To list the possible configuration parameters, use the /config command.

[clearloop]

now 12:11, waiting for 2 hrs see if the timeout logic still has bug xd

[clearloop]

f687051 requires rerun implementation #3088