Skip to content

Contributors fix#413

Open
sitjart wants to merge 5 commits intoELIXIR-Belgium:mainfrom
sitjart:contributors-fix
Open

Contributors fix#413
sitjart wants to merge 5 commits intoELIXIR-Belgium:mainfrom
sitjart:contributors-fix

Conversation

@sitjart
Copy link
Copy Markdown

@sitjart sitjart commented Mar 27, 2026

Hardcoded case in contributor-tiles-all.html breaks lowercase contributors.yaml

The template expects site.data.CONTRIBUTORS but data files are typically lowercase:
Expected: _data/contributors.yaml → site.data.contributors
Template: site.data.CONTRIBUTORS → nil
Fix: {%- assign contributors_lookup = site.data.contributors | default: site.data.CONTRIBUTORS -%}

Documentation does not warn of this requirement

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant