correct blossom trigger list format#4821
Conversation
Signed-off-by: YanxuanLiu <[email protected]>
|
/build |
|
Now I remember the issue, it's missing a comma after Yanxuan's id. Could you please add the comma instead? |
we prefer to use |
|
ok I'm sure why model zoo link is provided here as a reference. could you please help document for future reference? |
|
@wyli we use the newline pattern everywhere w/ blossom-ci, I guess the first time when adil add the file might not consider the issue |
|
@YanxuanLiu please help add the comment here maybe to describe how to correctly add a new ID |
Updated in description |
|
@YanxuanLiu please help file a following PR to verify the change after merge, many thanks! |
Description
Fix the issue
Cannot trigger pre-merge correctlyModifications:
Status
Ready/Work in progress/Hold
Types of changes
./runtests.sh -f -u --net --coverage../runtests.sh --quick --unittests --disttests.make htmlcommand in thedocs/folder.