Skip to content

fix: update README for clarity and versioning guidance#218

Merged
ChristophShyper merged 2 commits into
masterfrom
docs/clarity
May 22, 2026
Merged

fix: update README for clarity and versioning guidance#218
ChristophShyper merged 2 commits into
masterfrom
docs/clarity

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot commented May 22, 2026

📝 Brief description

fix: update README for clarity and versioning guidance
fix: add package write permission to auto-release workflow

💻 Commits

  • 4878145 - ChristophShyper - 2026-05-22 22:34:19
    | fix: add package write permission to auto-release workflow
    |
  • 71c10d4 - ChristophShyper - 2026-05-22 20:56:38
    fix: update README for clarity and versioning guidance

📁 Modified files

.github/workflows/auto-release-create.yml | 1 +
README.md | 7 +++++--
2 files changed, 6 insertions(+), 2 deletions(-)

⚠️ Additional information

  • Pushed to a branch with a proper name and provided proper commit message.
  • Provided a clear and concise description of what the issue is.

Check CONTRIBUTING.md and CODE_OF_CONDUCT.md for more information

@ChristophShyper ChristophShyper merged commit 0ca8a22 into master May 22, 2026
3 checks passed
@ChristophShyper ChristophShyper deleted the docs/clarity branch May 22, 2026 20:49
cherny-sh added a commit to ignitetech-group/action-pull-request that referenced this pull request May 26, 2026
This is a `-s ours` (strategy-ours) merge. It records `upstream/master`
at SHA c5a99ac as a second parent of our HEAD without changing a single
byte of our tree — the entire point is to update git's commit-reachability
graph so that GitHub's "X commits ahead / Y commits behind" comparison
between this fork and devops-infra/action-pull-request stops counting
those 13 upstream commits as "behind".

The actual content reconciliation already happened in PR #2 (commits
0b5126e + 832dfcc), where I selectively ported upstream's security and
feature improvements onto our gh-CLI-based fork after a security review:

  - repository / repository_path inputs (multi-repo support)
  - max_body_bytes / max_diff_lines (large-diff overflow handling)
  - GIT_CONFIG_GLOBAL isolation
  - repository regex + path-traversal validation
  - perl-based template substitution via scripts/replace-template-diff.sh
  - body-overflow split into managed PR comments
  - 7 new unit tests under tests/unit/

NOT merged from upstream (deliberately, per fork policy "no new third
parties"):

  - new .github/workflows/* that call devops-infra/.github reusable
    workflows or devops-infra/triglav with `secrets: inherit`
  - action.yml `image: docker://devopsinfra/...` (we keep `image:
    Dockerfile` so consumers build from our source)
  - Taskfile.scripts.yml dev-time refactor

The 13 upstream commits being recorded as ancestors:

  c5a99ac chore(release): prepare v1.2.1 (devops-infra#227)
  6aa878a feat: add Triglav as end-to-end test orchestrator (devops-infra#222)
  562d37b chore(release): prepare v1.2.0 (devops-infra#221)
  766f664 feat: add repository and repository_path inputs (devops-infra#220)
  86d8692 chore(release): prepare v1.1.3 (devops-infra#219)
  0ca8a22 fix: update README for clarity and versioning (devops-infra#218)
  eb4c48b chore(release): prepare v1.1.2 (devops-infra#217)
  5b6f69b fix: update Alpine package handling (devops-infra#216)
  ea294d9 feat: improve workflow linting (devops-infra#215)
  378b46d chore: update action-pull-request to v1.1.1 (devops-infra#212)
  a8b34a6 chore: update action-pull-request to v1.1.0 (devops-infra#211)
  0916b92 feat: harden action for handling large diffs (devops-infra#210)
  889cc9b feat: group GitHub Actions and Docker (devops-infra#209)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant