You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The check_xml_unique.py tool can notify a developer when duplicate standard names exist in the dictionary, which is good to avoid since having duplicate standard names introduces an ambiguity which could then cause problems downstream (for example when the CCPP framework tries to generate interface codes).
However, there is no guarantee that a developer will run this tool before pushing changes to the standard names dictionary, which means a duplicate standard name could sneak in on accident. To avoid this problem, a Github Action workflow should be created that runs check_xml_unique.py on the dictionary file (standard_names.xml) whenever a Pull Request is opened or updated in this repo, which then should "fail" whenever a duplicate name is found.
Finally, along with the duplicate name check itself, the check_xml_unique.py should also run validate_xml_file similar to write_standard_name_table.py in order to ensure xmllint is always run on the dictionary file as well (to avoid any basic syntax errors).
The text was updated successfully, but these errors were encountered:
The
check_xml_unique.py
tool can notify a developer when duplicate standard names exist in the dictionary, which is good to avoid since having duplicate standard names introduces an ambiguity which could then cause problems downstream (for example when the CCPP framework tries to generate interface codes).However, there is no guarantee that a developer will run this tool before pushing changes to the standard names dictionary, which means a duplicate standard name could sneak in on accident. To avoid this problem, a Github Action workflow should be created that runs
check_xml_unique.py
on the dictionary file (standard_names.xml
) whenever a Pull Request is opened or updated in this repo, which then should "fail" whenever a duplicate name is found.Finally, along with the duplicate name check itself, the
check_xml_unique.py
should also runvalidate_xml_file
similar towrite_standard_name_table.py
in order to ensurexmllint
is always run on the dictionary file as well (to avoid any basic syntax errors).The text was updated successfully, but these errors were encountered: