Merge pull request #90 from openconfig/report-pyang-warnings

Report pyang warnings as errors
openconfig · Oct 13, 2023 · e19e17b · e19e17b
2 parents fbdfaa6 + a605d29
commit e19e17b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/cmd_gen/main.go b/cmd_gen/main.go
@@ -138,8 +138,8 @@ script_options=(
 function run-dir() {
   declare prefix="$workdir"/"$1"=="$2"==
   shift 2
-  echo pyang "${options[@]}" "$@" > ${prefix}cmd
-  if ! $($cmd "${options[@]}" "${script_options[@]}" "$@" &> ${prefix}pass); then
+  echo pyang -W error "${options[@]}" "$@" > ${prefix}cmd
+  if ! $($cmd -W error "${options[@]}" "${script_options[@]}" "$@" &> ${prefix}pass); then
     mv ${prefix}pass ${prefix}fail
   fi
 }

diff --git a/cmd_gen/main_test.go b/cmd_gen/main_test.go
@@ -66,8 +66,8 @@ script_options=(
 function run-dir() {
   declare prefix="$workdir"/"$1"=="$2"==
   shift 2
-  echo pyang "${options[@]}" "$@" > ${prefix}cmd
-  if ! $($cmd "${options[@]}" "${script_options[@]}" "$@" &> ${prefix}pass); then
+  echo pyang -W error "${options[@]}" "$@" > ${prefix}cmd
+  if ! $($cmd -W error "${options[@]}" "${script_options[@]}" "$@" &> ${prefix}pass); then
     mv ${prefix}pass ${prefix}fail
   fi
 }
@@ -97,8 +97,8 @@ script_options=(
 function run-dir() {
   declare prefix="$workdir"/"$1"=="$2"==
   shift 2
-  echo pyang "${options[@]}" "$@" > ${prefix}cmd
-  if ! $($cmd "${options[@]}" "${script_options[@]}" "$@" &> ${prefix}pass); then
+  echo pyang -W error "${options[@]}" "$@" > ${prefix}cmd
+  if ! $($cmd -W error "${options[@]}" "${script_options[@]}" "$@" &> ${prefix}pass); then
     mv ${prefix}pass ${prefix}fail
   fi
 }