hashgraph / hedera-smart-contracts

Contains Hedera Smart Contract Service supporting files
Apache License 2.0
46 stars 54 forks source link

fix: skipped title check CI job for forked PRs #1000

Closed quiet-node closed 3 weeks ago

quiet-node commented 3 weeks ago

Description: Title check CI job fail for PRs that are from a forked repository due to the lack of the secret.GITHUB_TOKEN. This PR adds an option to skip the check if the PRs are from forked repos.

Related issue(s):

Fixes #999

Notes for reviewer:

Checklist

github-actions[bot] commented 3 weeks ago

Test Results

 16 files  ± 0   82 suites  +3   12m 1s :stopwatch: + 5m 53s 290 tests +38  284 :white_check_mark: +45  6 :zzz: ±0  0 :x:  - 7  299 runs  +23  292 :white_check_mark: +30  7 :zzz: ±0  0 :x:  - 7 

Results for commit 155e7f5c. ± Comparison against base commit 11ef9e9b.

This pull request removes 3 and adds 41 tests. Note that renamed tests count towards both. ``` "before all" hook in "Admin key set to contractId" ‑ TokenManagmentContract Test Suite Extended update token info and keys test suite Admin key set to contractId "before all" hook in "Admin key set to contractId" "before each" hook for "Should NOT burn insufficient amount of token" ‑ @OZERC1155Token Test Suite "before each" hook for "Should NOT burn insufficient amount of token" "before each" hook for "should be able to update fixed fee in HTS token" ‑ TokenManagmentContract Test Suite Update fees "before each" hook for "should be able to update fixed fee in HTS token" ``` ``` Should NOT allow a non-operator to transfer tokens to another account ‑ @OZERC1155Token Test Suite Should NOT allow a non-operator to transfer tokens to another account Should NOT burn insufficient amount of token ‑ @OZERC1155Token Test Suite Should NOT burn insufficient amount of token Should allow an operator to transfer a token to another account ‑ @OZERC1155Token Test Suite Should allow an operator to transfer a token to another account Should allow an operator to transfer tokens in batch to another account ‑ @OZERC1155Token Test Suite Should allow an operator to transfer tokens in batch to another account Should burn token in batch ‑ @OZERC1155Token Test Suite Should burn token in batch should be able to change ADMIN key to ECDSA_secp256k and perform admin action with same contract ‑ TokenManagmentContract Test Suite Extended update token info and keys test suite Admin key set to contractId Positive should be able to change ADMIN key to ECDSA_secp256k and perform admin action with same contract should be able to change FREEZE key to ECDSA_secp256k and freeze the token with the same account ‑ TokenManagmentContract Test Suite Extended update token info and keys test suite Admin key set to contractId Positive should be able to change FREEZE key to ECDSA_secp256k and freeze the token with the same account should be able to change PAUSE key to ECDSA_secp256k and pause the token with the same account ‑ TokenManagmentContract Test Suite Extended update token info and keys test suite Admin key set to contractId Positive should be able to change PAUSE key to ECDSA_secp256k and pause the token with the same account should be able to change WIPE key to ECDSA_secp256k and wipe the token with the same account ‑ TokenManagmentContract Test Suite Extended update token info and keys test suite Admin key set to contractId Positive should be able to change WIPE key to ECDSA_secp256k and wipe the token with the same account should be able to update fixed HTS fee and royalty fee in NFT ‑ TokenManagmentContract Test Suite Update fees should be able to update fixed HTS fee and royalty fee in NFT … ```