#87 Fix error message when container.yaml exists
Closed 5 years ago by nphilipp. Opened 5 years ago by otaylor.
modularity/ otaylor/fedmod container-yaml-exists  into  master

@@ -101,7 +101,7 @@ 

              if os.path.exists(output_modulemd):

                  raise click.ClickException(f"{output_modulemd} exists. Pass --force to overwrite")

              if os.path.exists(output_containerspec):

-                 raise click.ClickException(f"{output_modulemd} exists. Pass --force to overwrite")

+                 raise click.ClickException(f"{output_containerspec} exists. Pass --force to overwrite")

  

          super().run(output_modulemd)

  

When container.yaml existed, the error message mentioned '<appname.yaml>'
instead.

LGTM, obvious copy-paste error

Rebased and applied in commit 3a9a016.

Pull-Request has been closed by nphilipp

5 years ago

Fedora updates for 0.3.0 containing this change: F28, F29

Metadata