Closed biswassri closed 2 months ago
/lgtm
/retest
@biswassri: The following test failed, say /retest
to rerun all failed tests or /retest-required
to rerun all mandatory failed tests:
Test name | Commit | Details | Required | Rerun command |
---|---|---|---|---|
ci/prow/test-ai-edge | 57c9d3e914fc4b15d7678f32db6ae3555da6c5b8 | link | true | /test test-ai-edge |
Full PR test history. Your PR dashboard.
Rebase is needed
@StevenTobin @adelton I have made the changes according to your comments, if everything looks good can any of you mark it approved?
lgtm but @adelton's comments were more substantive than mine so I'll let him approve.
LGTM.
/approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jackdelahunt, MarianMacik
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/lgtm
JIRA: RHOAIENG-205
Description
Adding a glossary for stand terms. This is just the first set. We plan to iteratively update this terminology list. Ref: https://github.com/LaVLaS/ai-edge/blob/glossary/docs/glossary.md
I did move the doc that Landon had started but do we want to add more terminology to this?
How Has This Been Tested?
N/A
Merge criteria: