Skip to content

not using namespace in header files#1081

Merged
stcheng merged 2 commits intosonic-net:masterfrom
wendani:namespace_master
Oct 3, 2019
Merged

not using namespace in header files#1081
stcheng merged 2 commits intosonic-net:masterfrom
wendani:namespace_master

Conversation

@wendani
Copy link
Contributor

@wendani wendani commented Oct 3, 2019

What I did

Why I did it

How I verified it

Details if related

Signed-off-by: Wenda Ni <wenni@microsoft.com>
@wendani wendani requested review from lguohan and qiluo-msft October 3, 2019 02:41
@wendani wendani force-pushed the namespace_master branch 3 times, most recently from 920e4e8 to 641cf5d Compare October 3, 2019 03:02
Copy link
Contributor

@stcheng stcheng left a comment

Choose a reason for hiding this comment

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

@wendani wendani force-pushed the namespace_master branch 19 times, most recently from 8cd1f56 to bd3922a Compare October 3, 2019 21:44
Signed-off-by: Wenda Ni <wenni@microsoft.com>
@wendani wendani requested a review from stcheng October 3, 2019 22:00
@wendani
Copy link
Contributor Author

wendani commented Oct 3, 2019

Fixed compile error. #1081 (review)

@wendani
Copy link
Contributor Author

wendani commented Oct 3, 2019

retest this please

Copy link
Contributor

@stcheng stcheng left a comment

Choose a reason for hiding this comment

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

thanks

@stcheng stcheng merged commit 953474a into sonic-net:master Oct 3, 2019
EdenGri pushed a commit to EdenGri/sonic-swss that referenced this pull request Feb 28, 2022
* show_interface transceiver changes for multi-asic, takes care of sub commands
       eeprom    Show interface transceiver EEPROM information
       lpmode    Show interface transceiver low-power mode...
       presence  Show interface transceiver presence  
* Rearranging of common functions in config/main.py to py-common
* Updates to show command and testcases
oleksandrivantsiv pushed a commit to oleksandrivantsiv/sonic-swss that referenced this pull request Mar 1, 2023
- Update SAI VoQ support (sonic-net#1107) …
- Voq system (sonic-net#1081) …
- [meta] Add support for ignored enum values (sonic-net#1099)
- TPID SAI proposal (sonic-net#1089) …
- ACL GRE key match (sonic-net#1076) …
- Add IPv6 NS and NA Traps (sonic-net#1092) …
- MACsec flow list attribute added in MACsec object (sonic-net#1095) …
- Add Enterprise Number for IPFIX Report Type (sonic-net#1072) …
- Provide TTL and QoS treatment during MPLS encap and decap (sonic-net#1079)
- Create and Set for Tunnel Attributes (sonic-net#1086) …
oleksandrivantsiv pushed a commit to oleksandrivantsiv/sonic-swss that referenced this pull request Mar 1, 2023
Signed-off-by: Ze Gan <ganze718@gmail.com>
Janetxxx pushed a commit to Janetxxx/sonic-swss that referenced this pull request Nov 10, 2025
Signed-off-by: Wenda Ni <wenni@microsoft.com>
jianyuewu pushed a commit to jianyuewu/sonic-swss that referenced this pull request Dec 24, 2025
Add a checker to prevent adding new CFG_ macros when update common/schema.h.

Why I did it
CFG_XXX_TABLE_NAME macros should be auto-generated from YANG models via cfg_schema.h.

How I did it
Introduced .github/workflows/prevent-cfg.yml which runs on PRs that modify common/schema.h.
It extracts all CFG_ macros defined in the file and compares them against an allowed legacy list.

Work item tracking
Microsoft ADO: 35075048
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants