Page MenuHomeDevCentral

Read correctly empty configuration file
ClosedPublic

Authored by dereckson on Sep 25 2025, 17:00.
Tags
None
Referenced Files
F24608357: D3725.diff
Sun, Mar 1, 11:52
F24603512: D3725.diff
Sun, Mar 1, 07:04
F24602053: D3725.diff
Sun, Mar 1, 05:25
F24601980: D3725.id9633.diff
Sun, Mar 1, 05:24
F24600297: D3725.diff
Sun, Mar 1, 03:56
F24600274: D3725.id9634.diff
Sun, Mar 1, 03:55
F24596636: D3725.id9634.diff
Sun, Mar 1, 00:27
F24595048: D3725.id9634.diff
Sat, Feb 28, 23:01
Subscribers
None

Details

Summary

When a configuration file exists but is empty, YAML parser returns None.

This change returns an empty dict to represent the empty configuration.

This fixes the following issue for empty config file:

File "/home/dereckson/dev/python/general/lib64/python3.13/site-packages/mergedictionaries/app/app.py", line 118, in <lambda>
  self.context["config"].get("git", []), self.context["git"]
  ^^^^^^^^^^^^^^^^^^^^^^^^^^

AttributeError: 'NoneType' object has no attribute 'get'

Test Plan
  • create an empty configuration file
  • merge-dictionaries --extract

Diff Detail

Repository
rMD Merge dev dictionaries
Lint
Lint Errors
SeverityLocationCodeMessage
Errorsrc/mergedictionaries/app/app.py:66E501flake8 E501
Errorsrc/mergedictionaries/app/app.py:103E501flake8 E501
Unit
No Test Coverage
Branch
empty-git-config-issue
Build Status
Buildable 5973
Build 6255: arc lint + arc unit

Event Timeline

dereckson created this revision.
This revision is now accepted and ready to land.Sep 25 2025, 17:00
This revision was landed with ongoing or failed builds.Sep 25 2025, 17:01
This revision was automatically updated to reflect the committed changes.