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.

Sign up for GitHub

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

Jump to bottom

Update Cython / PyYAML #17297

Merged
merged 1 commit into from
Sep 19, 2023
Merged

Update Cython / PyYAML #17297

merged 1 commit into from
Sep 19, 2023

Conversation

churik
Copy link
Member

@churik churik commented Sep 15, 2023 •
edited
Loading

possible fix #17296

@churik churik added the tests label Sep 15, 2023
@churik churik self-assigned this Sep 15, 2023
@status-im-auto
Copy link
Member

status-im-auto commented Sep 15, 2023 •
edited
Loading

Jenkins Builds

Click to see older builds (8)
❔ Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 9adb2e0 #1 2023-09-15 10:31:04 ~6 min ios 📱ipa 📲
✔️ 9adb2e0 #1 2023-09-15 10:34:38 ~10 min android-e2e 🤖apk 📲
✔️ 9adb2e0 #1 2023-09-15 10:34:58 ~10 min android 🤖apk 📲
✔️ 9adb2e0 #1 2023-09-15 10:35:23 ~10 min tests 📄log
✔️ a9bd0d2 #2 2023-09-19 08:54:37 ~5 min android 🤖apk 📲
✔️ a9bd0d2 #2 2023-09-19 08:55:04 ~6 min android-e2e 🤖apk 📲
✔️ a9bd0d2 #2 2023-09-19 08:56:03 ~7 min ios 📱ipa 📲
✔️ a9bd0d2 #2 2023-09-19 08:58:25 ~9 min tests 📄log
❔ Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 1850fb3 #3 2023-09-19 10:44:12 ~5 min android-e2e 🤖apk 📲
✔️ 1850fb3 #3 2023-09-19 10:44:47 ~6 min android 🤖apk 📲
✔️ 1850fb3 #3 2023-09-19 10:45:42 ~7 min ios 📱ipa 📲
✔️ 1850fb3 #3 2023-09-19 10:47:07 ~8 min tests 📄log
✔️ 94a61da #4 2023-09-19 10:58:00 ~5 min android-e2e 🤖apk 📲
✔️ 94a61da #4 2023-09-19 10:58:37 ~6 min ios 📱ipa 📲
✔️ 94a61da #4 2023-09-19 11:01:18 ~9 min android 🤖apk 📲
✔️ 94a61da #4 2023-09-19 11:01:39 ~9 min tests 📄log

@status-im-auto
Copy link
Member

100% of end-end tests have passed

Total executed tests: 1
Failed tests: 0
Passed tests: 1

Passed tests (1)

Click to expand

Class TestCommunityOneDeviceMerged:

1. test_community_navigate_to_channel_when_relaunch, id: 702846
Device sessions

@churik churik requested a review from yevh-berdnyk September 19, 2023 10:37
@churik churik marked this pull request as ready for review September 19, 2023 10:37
@churik
Copy link
Member Author

churik commented Sep 19, 2023

Confirmed that it is installing all dependencies here

@churik churik changed the title WIP: add Cython Update Cython / PyYAML Sep 19, 2023
@churik churik merged commit 5d6800c into develop Sep 19, 2023
@churik churik deleted the tests/patch-pyyaml branch September 19, 2023 11:52
yakimant
yakimant reviewed Sep 19, 2023
View reviewed changes
test/appium/requirements.txt
@@ -1,7 +1,8 @@
Appium-Python-Client==2.11.1
Pillow==8.1.0
PyGithub==1.55
PyYAML==5.4
PyYAML==5.3.1
Cython<3.0
Copy link
Member

@yakimant yakimant Sep 19, 2023 •
edited
Loading

Choose a reason for hiding this comment

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

I think Cython<3.0 is not required, when lowering down the PyYAML version to 5.4.1

ibrkhalil pushed a commit that referenced this pull request Oct 1, 2023
@churik @ibrkhalil
e2e: update cython and pyyaml (#17297)
dae1fab
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yakimant yakimant yakimant left review comments

@yevh-berdnyk yevh-berdnyk yevh-berdnyk approved these changes

Assignees

@churik churik

Labels
tests
Projects
Pipeline for QA
Archived in project
Legacy QA Automation Project
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Re-run e2e job is failing
4 participants
@churik @status-im-auto @yakimant @yevh-berdnyk