Skip to content

chore: add SAP security policy and warn about ex6 typo#35

Merged
jung-thomas merged 1 commit into
mainfrom
chore/security-and-issue-32
Jun 12, 2026
Merged

chore: add SAP security policy and warn about ex6 typo#35
jung-thomas merged 1 commit into
mainfrom
chore/security-and-issue-32

Conversation

@jung-thomas

Copy link
Copy Markdown
Contributor

Summary

Two small but unrelated improvements bundled together (both spawned from a project-health review):

1. SECURITY.md (new)

Canonical SAP-samples security policy (V0.0.1 from SAP-samples/.github). Points to the SAP Trust Center for vulnerability reports. The version header lets SAP's tooling auto-update it.

2. WARNING callout in exercises/ex6/README.mdfixes #32

Two screenshots in the upstream SAP tutorial (hana-cloud-cap-calc-view) show V_INTERATION (missing the second C) instead of the correct V_INTERACTION — affecting create_calc_view.png and folder_view.png.

We can't pixel-edit the upstream PNGs, so this PR adds a > [!WARNING] admonition right after the tutorial link in our exercise README. Participants will see the heads-up before hitting the typo'd screenshots. The issue reporter explicitly said they trust screenshots more than text — this preempts that trust gap.

A separate upstream issue will be filed against sap-tutorials/Tutorials so the screenshots get refreshed; once that's done the callout can be removed.

Context (off-PR)

This branch is also our first PR through the new branch protection ruleset (replaces the legacy classic protection — rule require_last_push_approval is now satisfied as required by OSPO before 2026-06-16).

Closes #32

- Add canonical SAP-samples SECURITY.md (V0.0.1, points to SAP Trust Center)
- Add WARNING callout in exercises/ex6/README.md flagging upstream typo
  V_INTERATION -> V_INTERACTION in two screenshots of the linked tutorial.
  Fixes #32.
@jung-thomas jung-thomas merged commit 418036c into main Jun 12, 2026
4 checks passed
@jung-thomas jung-thomas deleted the chore/security-and-issue-32 branch June 12, 2026 15:17
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.

Excercise 6 screenshot: change V_INTERATION to V_INTERACTION

1 participant