Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SYS-164 added workflow to run linter and trigger smoke tests #1

Merged
merged 3 commits into from
May 17, 2024

Conversation

TieDyedSheep
Copy link

@TieDyedSheep TieDyedSheep commented May 15, 2024

Added a github action yaml to to basic builds, linting, test the patches can be applied and finally trigger the Jenkins smoke test job. If no patches exist, the job will skip that step. if the build, linting, formatter or patches fail, the workflow will fail.

@TieDyedSheep TieDyedSheep requested a review from a team May 15, 2024 19:40
Copy link

linear bot commented May 15, 2024

@TieDyedSheep TieDyedSheep force-pushed the SYS-164-basic-github-build-pipeline branch 4 times, most recently from 755c06c to ade1042 Compare May 15, 2024 21:33
muni-corn
muni-corn previously approved these changes May 15, 2024
@muni-corn
Copy link

Looks good to me!

@TieDyedSheep TieDyedSheep force-pushed the SYS-164-basic-github-build-pipeline branch from 9452be8 to 682274d Compare May 15, 2024 22:50
@TieDyedSheep
Copy link
Author

This is the Jenkins job I trigger, it was provided parameters to override the shardus_types parameters with the branch from the PR and this triggered it's own action and ran successfully
https://devs-work.shardeum.org/job/Testing/job/Smoke/job/smoke_10_plus_10/113/
https://github.com/shardeum/lib-types/actions/runs/9103671559

@TieDyedSheep
Copy link
Author

This was approved and the approval was removed when I resolved a conversation so I'm just bypassing and merging this

@TieDyedSheep TieDyedSheep merged commit be08f93 into dev May 17, 2024
4 checks passed
@TieDyedSheep TieDyedSheep deleted the SYS-164-basic-github-build-pipeline branch May 21, 2024 15:11
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.

3 participants