Skip to content

fix: use dc:creator instead of author for item authors - #455

Open
Zerthick wants to merge 1 commit into
Guts:mainfrom
Zerthick:fix/dc-creator-for-authors
Open

fix: use dc:creator instead of author for item authors#455
Zerthick wants to merge 1 commit into
Guts:mainfrom
Zerthick:fix/dc-creator-for-authors

Conversation

@Zerthick

Copy link
Copy Markdown

Description

Use <dc:creator> instead of <author> for item authors in the RSS 2.0 feed.

The RSS 2.0 spec requires <author> to be an email address. Human-readable names should use <dc:creator> from the Dublin Core namespace (which is already declared in the feed).

Changes

  • Change <author> to <dc:creator> in rss.xml.jinja2 template
  • Update XSL HTML preview to read dc:creator (items) and managingEditor (channel)
  • Add test verifying <dc:creator> usage and absence of <author> in items

Split from: #451


💡 AI Contribution Level: 8 — Based on the VisiData AI contribution scale, this was an AI-assisted contribution where the AI performed the code splitting and PR creation with minimal human guidance.

RSS 2.0 spec requires <author> to be an email address; the dc namespace
is already declared and <dc:creator> is the correct element for names.

- Change <author> to <dc:creator> in rss.xml.jinja2 template
- Update XSL HTML preview to read dc:creator and managingEditor
- Add test verifying dc:creator usage
@github-actions github-actions Bot added bug Something isn't working quality Tests, project resiliency, etc. labels Aug 10, 2026
@sonarqubecloud

Copy link
Copy Markdown

@codecov

codecov Bot commented Aug 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.16%. Comparing base (1b5e630) to head (7c34564).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #455   +/-   ##
=======================================
  Coverage   83.16%   83.16%           
=======================================
  Files          11       11           
  Lines         784      784           
  Branches      133      133           
=======================================
  Hits          652      652           
  Misses         86       86           
  Partials       46       46           
Flag Coverage Δ
unittests 83.16% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working quality Tests, project resiliency, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant