Fix bug in dependency update workflow. (#685)

* Fix bug with logfile if --skip_android is set. * Fix workflow to handle missing logfile.

Author: jonsimantov

.github/workflows/update-dependencies.yml

@@ -87,13 +87,16 @@ jobs:
  echo "::error ::Neither Android nor iOS selected. Exiting."
  exit 1
  fi
+ touch ${UPDATE_LOGFILE} # In case it wasn't generated before
+ cat ${UPDATE_LOGFILE}
 
  - name: Push branch if there are changes
  id: push-branch
  run: |
  if ! git update-index --refresh; then
  # Do a bit of post-processing on the update log to split it by platform.
  UPDATE_LOGFILE_PROCESSED=update_log_processed.txt
+ touch "${UPDATE_LOGFILE_PROCESSED}"
  if grep -q ^Android: "${UPDATE_LOGFILE}"; then
  echo "### Android" >> "${UPDATE_LOGFILE_PROCESSED}"
  echo "" >> "${UPDATE_LOGFILE_PROCESSED}"

scripts/update_android_ios_dependencies.py

@@ -738,6 +738,8 @@ def parse_cmdline_args():
 logfile_lines = set()
 
 def main():
+ global logfile_lines
+
  args = parse_cmdline_args()
  # Readme files have to be updated for both Android and iOS dependencies.
  readme_files = get_files(args.readmefiles, file_extension='.md',
@@ -772,9 +774,9 @@ def main():
  for gradle_file in gradle_files:
  modify_gradle_file(gradle_file, latest_android_versions_map, args.dryrun)
 
-  if args.logfile:
-  with open(args.logfile, 'w') as logfile_file:
-  logfile_file.write("\n".join(sorted(list(logfile_lines))) + "\n")
+ if args.logfile:
+ with open(args.logfile, 'w') as logfile_file:
+ logfile_file.write("\n".join(sorted(list(logfile_lines))) + "\n")
 
 if __name__ == '__main__':
  main()