Skip to content

docs: add BMWK-EU funding notice to README#158

Merged
nexus49 merged 1 commit intomainfrom
docs/add-bmwk-eu-funding-notice
Mar 5, 2026
Merged

docs: add BMWK-EU funding notice to README#158
nexus49 merged 1 commit intomainfrom
docs/add-bmwk-eu-funding-notice

Conversation

@nexus49
Copy link
Member

@nexus49 nexus49 commented Mar 5, 2026

Summary

  • Adds the BMWK-EU (Bundesministerium für Wirtschaft und Klimaschutz) funding notice to the end of the README as required for all public repositories.

Summary by CodeRabbit

  • Documentation
    • Added funding logo to project documentation.

Signed-off-by: Bastian Echterhölter <bastian.echterhoelter@sap.com>
On-behalf-of: @SAP <bastian.echterhoelter@sap.com>
@nexus49 nexus49 requested a review from a team as a code owner March 5, 2026 10:50
@github-actions github-actions bot added documentation Improvements or additions to documentation fix labels Mar 5, 2026
@coderabbitai
Copy link

coderabbitai bot commented Mar 5, 2026

📝 Walkthrough

Walkthrough

README.md has been updated with the addition of a centered HTML image tag displaying the BMWE-EU funding logo after the Licensing section. This is purely a documentation content addition with no functional behavior changes.

Changes

Cohort / File(s) Summary
Documentation
README.md
Added centered HTML image tag for BMWE-EU funding logo after Licensing section (+3 lines).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A logo hops into view,
Centered with care, crisp and true,
In markdown's embrace, it shall stay,
Funding's badge brightens the day! 🎀

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and accurately summarizes the main change: adding a funding notice to the README documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch docs/add-bmwk-eu-funding-notice

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🧹 Nitpick comments (1)
README.md (1)

57-57: Consider avoiding external hotlinking for a required compliance logo.

For a mandatory notice, relying on a third-party URL can create a future broken/compliance gap if that asset moves. Prefer storing the logo in-repo (or an official, stable host controlled by your org) and referencing it locally.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@README.md` at line 57, Replace the external hotlinked logo in README.md by
committing the image into the repository (e.g., assets/img/BMWK-EU.png) and
update the <img> tag’s src to the local path; specifically locate the <img>
element with alt="Bundesministerium für Wirtschaft und Energie (BMWE)-EU funding
logo" and change its src from the external URL to the new in-repo relative path
so the required compliance logo is served locally and won’t break if the
external host changes.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@README.md`:
- Line 57: Update the image alt text to use the correct funding body
name/acronym: replace the current alt value "Bundesministerium für Wirtschaft
und Energie (BMWE)-EU funding logo" with "Bundesministerium für Wirtschaft und
Klimaschutz (BMWK-EU) funding logo" in the <img> tag so the alt attribute
matches the PR requirement and required wording.

---

Nitpick comments:
In `@README.md`:
- Line 57: Replace the external hotlinked logo in README.md by committing the
image into the repository (e.g., assets/img/BMWK-EU.png) and update the <img>
tag’s src to the local path; specifically locate the <img> element with
alt="Bundesministerium für Wirtschaft und Energie (BMWE)-EU funding logo" and
change its src from the external URL to the new in-repo relative path so the
required compliance logo is served locally and won’t break if the external host
changes.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: dfa9b897-b0a4-467d-a140-6b9945491fb7

📥 Commits

Reviewing files that changed from the base of the PR and between 7ad14b9 and cb7fdf7.

📒 Files selected for processing (1)
  • README.md




<p align="center"><img alt="Bundesministerium für Wirtschaft und Energie (BMWE)-EU funding logo" src="https://apeirora.eu/assets/img/BMWK-EU.png" width="400"/></p>
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major

Fix funding body name/acronym mismatch in the notice text.

At Line 57, the notice says BMWE and “Wirtschaft und Energie,” but this PR’s requirement is BMWK-EU (Bundesministerium für Wirtschaft und Klimaschutz). Please align the alt text with the required wording to avoid compliance/documentation drift.

Proposed fix
-<p align="center"><img alt="Bundesministerium für Wirtschaft und Energie (BMWE)-EU funding logo" src="https://apeirora.eu/assets/img/BMWK-EU.png" width="400"/></p>
+<p align="center"><img alt="Bundesministerium für Wirtschaft und Klimaschutz (BMWK)-EU funding logo" src="https://apeirora.eu/assets/img/BMWK-EU.png" width="400"/></p>
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
<p align="center"><img alt="Bundesministerium für Wirtschaft und Energie (BMWE)-EU funding logo" src="https://apeirora.eu/assets/img/BMWK-EU.png" width="400"/></p>
<p align="center"><img alt="Bundesministerium für Wirtschaft und Klimaschutz (BMWK)-EU funding logo" src="https://apeirora.eu/assets/img/BMWK-EU.png" width="400"/></p>
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@README.md` at line 57, Update the image alt text to use the correct funding
body name/acronym: replace the current alt value "Bundesministerium für
Wirtschaft und Energie (BMWE)-EU funding logo" with "Bundesministerium für
Wirtschaft und Klimaschutz (BMWK-EU) funding logo" in the <img> tag so the alt
attribute matches the PR requirement and required wording.

@nexus49 nexus49 merged commit f9b4ffe into main Mar 5, 2026
9 checks passed
@nexus49 nexus49 deleted the docs/add-bmwk-eu-funding-notice branch March 5, 2026 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant