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

CR-1221546, CR-1219147, CR-1219468 #8732

Merged
merged 2 commits into from
Feb 1, 2025
Merged

CR-1221546, CR-1219147, CR-1219468 #8732

merged 2 commits into from
Feb 1, 2025

Conversation

AShivangi
Copy link
Collaborator

Problem solved by the commit

  • CR-1221546 xrt-smi examine: should indicate missing flag argument
  • CR-1219147 Space is missing b/w xrt-smi and [--help] in usage
  • CR-1219468 xrt-smi examine -f JSON issues

Bug / issue (if any) fixed, which PR introduced the bug, how it was discovered

DSV testing

How problem was solved, alternative solutions (if any) and why they were rejected

  • First CR was fixed by adding some DRC checks in examine as well as validate for consistency purposes
  • Added space
  • Similar to first CR. Made similar changes in validate for consistency

Risks (if any) associated the changes in the commit

N/A

What has been tested and how, request additional testing if necessary

Tested multiple commands on linux and windows

Documentation impact (if any)

N/A

@stsoe stsoe merged commit 12c637d into Xilinx:master Feb 1, 2025
20 checks passed
@AShivangi AShivangi deleted the CRs branch February 4, 2025 16:45
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.

3 participants