Hadron / Hadron/carthage

Empty configuration file raises AssertionError

Open
#131 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
11
Forks
8
Avg merge
9h 26m
Merged PRs (30d)
6

Description

Pointing carthage-runner to empty config.yml raises
```
Traceback (most recent call last):
File "/root/carthage/bin/carthage-runner", line 192, in
args, unknown = carthage.utils.carthage_main_setup(parser, unknown_ok=True)
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^
File "/root/carthage/carthage/utils.py", line 285, in carthage_main_setup
config.load_yaml(f, ignore_import_errors=ignore_import_errors)
~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/root/carthage/carthage/config/layout.py", line 94, in load_yaml
assert isinstance(d, dict)
~~~~~~~~~~^^^^^^^^^
AssertionError
```
This could have a clearer message.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with carthage/config/layout.py at load_yaml, then reproduce the failure by pointing carthage-runner at an empty config.yml. Replace the AssertionError outcome with a clearer configuration error, and verify that the empty-file invocation reports the new message.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
infrastructure
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
52/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.