Check for empty description when converting metadata

[intentionally-left-nil]

To prevent an exception, check for an empty description Fixes: #645

Test plan: Run the unit tests without the fix -> Fails with an exception. Passes after the fix

[intentionally-left-nil]

There's no specified behavior of what the description should be if empty - To match the existing behavior as much as possible (where an extra newline is appended), I did the same in this PR. Let me know if you'd like a different default value

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.25%. Comparing base (426b8dd) to head (6f7bdcd).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #646      +/-   ##
==========================================
+ Coverage   78.21%   78.25%   +0.03%     
==========================================
  Files          14       14              
  Lines        1120     1122       +2     
==========================================
+ Hits          876      878       +2     
  Misses        244      244              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[agronholm]

Please add a changelog entry (in docs/news.rst)

[agronholm]

Looks like there was a mishap with syntax. If you'll just sort that out and add the changelog entry, we're good for merging.

[agronholm]

Please don't force push. You just overwrote my edits to news.rst.

[intentionally-left-nil]

Ah sorry about that :) Different habits for fixing issues. I don't have your version in my reflog unfortunately. Did you want to take a stab at fixing the news.rst to your liking?

[agronholm]

I've pushed my edits now.

[agronholm]

Thanks.