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

ci: extrac Lint and Tests to separate job #13862

Merged
merged 1 commit into from
Aug 26, 2022
Merged

ci: extrac Lint and Tests to separate job #13862

merged 1 commit into from
Aug 26, 2022

Conversation

jakubgs
Copy link
Member

@jakubgs jakubgs commented Aug 26, 2022

This way we can make PRs depend only on successful tests, and not whole builds for all platforms, which take 10 min or more.

The added benefit is that builds are slightly faster, and test logs can be publicly available.

@jakubgs jakubgs added the tests label Aug 26, 2022
@jakubgs jakubgs requested a review from a team August 26, 2022 13:21
@jakubgs jakubgs self-assigned this Aug 26, 2022
@status-github-bot status-github-bot bot added this to REVIEW in Pipeline for QA Aug 26, 2022
@status-im-auto
Copy link
Member

status-im-auto commented Aug 26, 2022

Jenkins Builds

Click to see older builds (1)
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 72eb9dd #3 2022-08-26 13:27:08 ~2 min tests 📦log
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ a9defa3 #4 2022-08-26 13:33:25 ~4 min tests 📦log
✔️ a9defa3 #3 2022-08-26 13:37:04 ~7 min android-e2e 📦apk 📲
✔️ a9defa3 #3 2022-08-26 13:37:25 ~8 min android 📦apk 📲
✔️ a9defa3 #3 2022-08-26 13:44:42 ~15 min ios 📦ipa 📲
✔️ 0a4d181 #5 2022-08-26 13:50:24 ~2 min tests 📦log
✔️ 0a4d181 #4 2022-08-26 13:55:53 ~7 min android 📦apk 📲
✔️ 0a4d181 #4 2022-08-26 13:55:56 ~7 min android-e2e 📦apk 📲
✔️ 0a4d181 #4 2022-08-26 14:01:19 ~13 min ios 📦ipa 📲

This way we can make PRs depend only on successful tests, and not whole
builds for all platforms, which take 10 minutes or more.

Signed-off-by: Jakub Sokołowski <jakub@status.im>
@jakubgs jakubgs marked this pull request as ready for review August 26, 2022 14:04
@jakubgs jakubgs merged commit 0a4d181 into develop Aug 26, 2022
Pipeline for QA automation moved this from REVIEW to DONE Aug 26, 2022
@jakubgs jakubgs deleted the ci/tests-job branch August 26, 2022 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

3 participants