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

Add known error for yast2_detect_logs #16277

Merged
merged 1 commit into from Feb 7, 2023
Merged

Add known error for yast2_detect_logs #16277

merged 1 commit into from Feb 7, 2023

Conversation

sofiasyria
Copy link
Contributor

@sofiasyria sofiasyria commented Jan 24, 2023 •
edited

@sofiasyria sofiasyria added WIP Work in progress qe-yam labels Jan 24, 2023
@github-actions
Copy link

github-actions bot commented Jan 24, 2023 •
edited by rakoenig

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.

@sofiasyria sofiasyria changed the title [WIP] Add known error for yast2_detect_logs Add known error for yast2_detect_logs Jan 24, 2023
@sofiasyria sofiasyria removed the WIP Work in progress label Jan 24, 2023
rakoenig
rakoenig reviewed Jan 24, 2023
View reviewed changes
Copy link
Contributor

@rakoenig rakoenig left a comment

Choose a reason for hiding this comment

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

LGTM

@sofiasyria sofiasyria added the WIP Work in progress label Jan 25, 2023
@sofiasyria sofiasyria removed the WIP Work in progress label Jan 27, 2023
@jknphy jknphy merged commit 9da50b7 into os-autoinst:master Feb 7, 2023
@sofiasyria sofiasyria deleted the y2logs branch April 6, 2023 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rakoenig rakoenig rakoenig left review comments

Assignees

@JRivrain JRivrain

Labels
qe-yam
Projects
None yet
Milestone
No milestone
4 participants
@sofiasyria @rakoenig @jknphy @JRivrain