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

Fix testrefs in lists #5376

Merged
merged 1 commit into from
Nov 25, 2023
Merged

Fix testrefs in lists #5376

merged 1 commit into from
Nov 25, 2023

Conversation

asdil12
Copy link
Member

@asdil12 asdil12 commented Nov 24, 2023 •
edited
Loading

image

@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 24, 2023 •
edited by asdil12
Loading

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

Files matching assets/stylesheets/**:

  • Consider providing screenshots in a PR comment to show the difference visually

This is an automatically generated QA checklist based on modified files.

okurz
okurz reviewed Nov 24, 2023
View reviewed changes
Copy link
Member

@okurz okurz left a comment

Choose a reason for hiding this comment

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

removing code sounds good but I would need more context to be able to approve that. Maybe somebody else can approve nevertheless

@mergify mergify bot merged commit 872b397 into os-autoinst:master Nov 25, 2023
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@okurz okurz okurz left review comments

@Martchus Martchus Martchus approved these changes

@perlpunk perlpunk perlpunk approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@asdil12 @perlpunk @okurz @Martchus