Skip to content

sort tests_mark_conditions.yaml#6336

Merged
XuChen-MSFT merged 1 commit intosonic-net:masterfrom
XuChen-MSFT:sort-tests_mark_conditions.yaml
Sep 17, 2022
Merged

sort tests_mark_conditions.yaml#6336
XuChen-MSFT merged 1 commit intosonic-net:masterfrom
XuChen-MSFT:sort-tests_mark_conditions.yaml

Conversation

@XuChen-MSFT
Copy link
Copy Markdown
Contributor

Description of PR

Summary: fix the order of the yaml file "tests_mark_conditions.yaml"
Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 201911
  • 202012

Approach

What is the motivation for this PR?

keep the test cases in alphabetical order in "tests/common/plugins/conditional_mark/tests_mark_conditions.yaml"

How did you do it?

sort test cases in alphabetical order

How did you verify/test it?

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla bot commented Sep 13, 2022

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: XuChen-MSFT / name: Xu Chen (98e5ee2)

Copy link
Copy Markdown
Collaborator

@StormLiangMS StormLiangMS left a comment

Choose a reason for hiding this comment

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

LGTM

@yutongzhang-microsoft
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@XuChen-MSFT
Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@XuChen-MSFT XuChen-MSFT merged commit 836854b into sonic-net:master Sep 17, 2022
Azarack pushed a commit to Azarack/sonic-mgmt that referenced this pull request Oct 17, 2022
allen-xf pushed a commit to allen-xf/sonic-mgmt that referenced this pull request Oct 28, 2022
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