Repository: reanahub/reana-client

Issue:
Improve log message when gitignore is detected
When `.gitignore` is detected, currently user sees log message below:

```[2022-05-17T14:32:25] reana-demo-root6-roofit: reana-client upload -w root6-roofit-yadage-kubernetes
==> SUCCESS: Detected .gitignore file. Some files might get ignored.
==> SUCCESS: File /code/gendata.C was successfully uploaded.
```

It feels a bit weird to see "SUCCESS" for gitignore detection to be on the same level as "SUCCESS" when uploading `gendata.C`. It might be better to say "INFO" for messages like Detected gitignore, reanaignore, etc...

https://github.com/reanahub/reana-client/blob/7655cb032e9d42f7775525d2f1c53382d865e92a/reana_client/cli/files.py#L341-L344

Discussion:

Patch:
diff --git a/reana_client/cli/files.py b/reana_client/cli/files.py
index e434978..87b14db 100644
--- a/reana_client/cli/files.py
+++ b/reana_client/cli/files.py
@@ -340,7 +340,7 @@ def upload_files(  # noqa: C901
                 gitignore_spec = pathspec.PathSpec.from_lines("gitwildmatch", f)
             display_message(
                 "Detected .gitignore file. Some files might get ignored.",
-                msg_type="success",
+                msg_type="info",
             )
         except FileNotFoundError:
             return directories


Test patch:
diff --git a/tests/test_cli_files.py b/tests/test_cli_files.py
index 6b4f8fc..1eb85ca 100644
--- a/tests/test_cli_files.py
+++ b/tests/test_cli_files.py
@@ -271,7 +271,7 @@ def test_upload_file_respect_gitignore(get_workflow_specification_with_directory
                 )
 
                 assert (
-                    "Detected .gitignore file. Some files might get ignored."
+                    "==> Detected .gitignore file. Some files might get ignored."
                     in result.output
                 )
                 assert "test" not in result.output