Skip to content

sc 126: update GitHub ci to run scripts#406

Merged
Zitzak merged 1 commit intotemp_devfrom
marving/sc-126-update-github-ci-to-run-scripts
Feb 6, 2024
Merged

sc 126: update GitHub ci to run scripts#406
Zitzak merged 1 commit intotemp_devfrom
marving/sc-126-update-github-ci-to-run-scripts

Conversation

@Zitzak
Copy link
Collaborator

@Zitzak Zitzak commented Feb 2, 2024

What has been done?

  • Update CI to include testing scripts

Issue

https://linear.app/inverter/issue/SC-126/update-github-ci-to-run-scripts

@Zitzak Zitzak force-pushed the marving/sc-126-update-github-ci-to-run-scripts branch from c99b8f3 to 09f912c Compare February 2, 2024 04:42
@Zitzak Zitzak requested review from 0xNuggan and FHieser February 2, 2024 04:43
@Zitzak Zitzak force-pushed the marving/sc-126-update-github-ci-to-run-scripts branch from 09f912c to d9f4a2b Compare February 2, 2024 04:46
@Zitzak Zitzak changed the base branch from main to openZeppelinV5 February 2, 2024 04:47
@Zitzak Zitzak changed the base branch from openZeppelinV5 to temp_dev February 2, 2024 10:28
@Zitzak Zitzak force-pushed the marving/sc-126-update-github-ci-to-run-scripts branch from d9f4a2b to 64f4939 Compare February 6, 2024 13:37
Copy link
Contributor

@FHieser FHieser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good although I dont know what the token decimal changes do there

@Zitzak Zitzak force-pushed the marving/sc-126-update-github-ci-to-run-scripts branch from 64f4939 to ebfd352 Compare February 6, 2024 13:40
@Zitzak
Copy link
Collaborator Author

Zitzak commented Feb 6, 2024

Rebased it to only include the changes for the CI @FHieser

@Zitzak Zitzak merged commit 0cb5fd5 into temp_dev Feb 6, 2024
@Zitzak Zitzak deleted the marving/sc-126-update-github-ci-to-run-scripts branch February 6, 2024 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants