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

Fix thunderbird test issues due to package update #15551

Merged
merged 1 commit into from Sep 22, 2022

Conversation

rfan1
Copy link
Contributor

@rfan1 rfan1 commented Sep 19, 2022

poo#116524

qam-mail-thunderbird
qam-regression-message

@github-actions
Copy link

Great PR! Please pay attention to the following items before merging:

Files matching lib/**.pm:

  • Consider adding or extending unit tests in t/

This is an automatically generated QA checklist based on modified files

@a-kpappas a-kpappas self-requested a review September 19, 2022 09:01
@rfan1
Copy link
Contributor Author

rfan1 commented Sep 20, 2022

@a-kpappas
CI check passed, could you please help review again?

@rfan1
Copy link
Contributor Author

rfan1 commented Sep 22, 2022

@paolostivanin @martinsmarcelo could you please help review my PR?

@GraceWang571
Copy link
Contributor

I saw the verification runs covered 15SP3. Could you please verify if the PR works on 15SP4? Because since 15SP4 the GNOME version got updated to 41.2 not 3.xx any more.

@rfan1
Copy link
Contributor Author

rfan1 commented Sep 22, 2022

I saw the verification runs covered 15SP3. Could you please verify if the PR works on 15SP4? Because since 15SP4 the GNOME version got updated to 41.2 not 3.xx any more.

We don't run the tests on 15sp4 MU, however, I can pass the tests for GA https://openqa.suse.de/tests/9562540#

@paolostivanin
Copy link
Contributor

LGTM

1 similar comment
@GraceWang571
Copy link
Contributor

LGTM

@GraceWang571 GraceWang571 merged commit c6aedfd into os-autoinst:master Sep 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants