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

[Build][202012]: fix some version info missing in version control files issue (#10211) #13673

Merged

Conversation

xumia
Copy link
Collaborator

@xumia xumia commented Feb 6, 2023

Why I did it

Fix some version info missing in version control files issue

How I did it

Change to collect version before it cleaned up

How to verify it

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211

Description for the changelog

Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

…sonic-net#10211)

[Build]: fix some version info missing in version control files issue
@xumia xumia added the Build label Feb 6, 2023
@xumia xumia changed the title [Build]: fix some version info missing in version control files issue (#10211) [Build][202012]: fix some version info missing in version control files issue (#10211) Feb 6, 2023
@liuh-80
Copy link
Contributor

liuh-80 commented Feb 6, 2023

This is cherry-pick for PR: #10211

@liuh-80 liuh-80 requested review from liushilongbuaa and removed request for qiluo-msft February 9, 2023 01:26
@liuh-80
Copy link
Contributor

liuh-80 commented Feb 9, 2023

@liushilongbuaa , please review this PR, because you are the original reviewer for #10211

@qiluo-msft qiluo-msft merged commit 8657fba into sonic-net:202012 Feb 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants