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

Improve job definition layout and contents in a handful of dirs in the base providers (BugFix) #1088

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

kissiel
Copy link
Contributor

@kissiel kissiel commented Mar 19, 2024

Another portion of changes done by the pxu-fixer.
I added the hunks by hand to do a front-line review.

The changes proposed here are/should be only cosmetic.

Copy link
Collaborator

@Hook25 Hook25 left a comment

Choose a reason for hiding this comment

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

Just one requested change (remove the duplicated field)

providers/base/units/acpi/jobs.pxu Outdated Show resolved Hide resolved
providers/base/units/benchmarks/jobs.pxu Outdated Show resolved Hide resolved
providers/base/units/benchmarks/jobs.pxu Outdated Show resolved Hide resolved
providers/base/units/benchmarks/jobs.pxu Outdated Show resolved Hide resolved
providers/base/units/benchmarks/jobs.pxu Outdated Show resolved Hide resolved
providers/base/units/disk/encryption.pxu Outdated Show resolved Hide resolved
providers/base/units/disk/jobs.pxu Outdated Show resolved Hide resolved
providers/base/units/disk/jobs.pxu Outdated Show resolved Hide resolved
Tests that the systems wireless hardware can connect to a router using WPA
security and the 802.11b/g protocols after the system has been suspended.
_purpose: Tests that the system's wireless hardware can connect to a router using WPA security and the 802.11b/g protocols after the system has been suspended.
_summary: Ensure wireless hardware can connect using WPA and 802.11b/g after suspend.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
_summary: Ensure wireless hardware can connect using WPA and 802.11b/g after suspend.

Duplicated

5. Connect a Thunderbolt monitor to the docking station.
6. Connect a Thunderbolt3 storage to the docking station.
7. Connect a USB Type-C storage to the docking station.
8. Connect a USB3.0 storage to the docking station.
9. Connect a headset to the docking station.
_verification:
Mark it Pass if you have connected all I/O ports as mentioned above.
_summary:
Verify the initial setup of all I/O ports on the docking station by connecting various devices and ensure they meet the dock's specification.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Verify the initial setup of all I/O ports on the docking station by connecting various devices and ensure they meet the dock's specification.
Prepare all I/O ports on the dock by connecting all devices it supports.

Summary is wrong and long

Applied Max's suggestions (except one)

Co-authored-by: Massimiliano <massimiliano.girardi@canonical.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants