[doc]: Update README.md#1122
[doc]: Update README.md#1122kcudnik merged 7 commits intosonic-net:masterfrom mdanish-kh:update-docs
Conversation
|
@kcudnik Can you please review? |
|
not sure why those builds fail, it's just readme file |
|
/AzurePipelines run Azure.sonic-sairedis |
|
Commenter does not have sufficient privileges for PR 1122 in repo sonic-net/sonic-sairedis |
|
Closed and re-opened the PR to re-run the pipelines, but it is still failing. |
|
The unit tests are repeatedly failing. |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
can you try to do a force push on this branch with empty commit to retriger all build ? |
@kcudnik I believe closing and re-opening the PR does the same. It re-runs the pipelines. I have been doing this for a few days, everytime the checks fail on the unittests in pipelines. But I'll try to push an empty commit as well. |
|
this is wired since other PR's are passing on tests, for example this one #1132 which was posted later |
The same test i.e., |
|
here: please fix whitespaces :) |
|
@kcudnik Thanks for pointing this out! Didn't realize a trailing white space causes issues in the pipelines, will keep this in mind in the future. All checks have passed now. Could you please review again? |
What I did