[SUP-7562] Adjust VERSION_SPEC

[adam-vessey]

It appears that dispatched workflows don't expose things in quite the same way.

Summary by CodeRabbit

• Chores
  • Standardized workflow configuration formatting with consistent line endings
  • Enhanced package version selection logic to dynamically vary based on workflow trigger type and Git reference type, differentiating between manual dispatches and automated executions

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

Updated workflow file formatting with Windows-style line endings and YAML normalization, and modified composer package version specification logic to conditionally use different GitHub references based on event type (workflow_dispatch vs. others).

Changes

Cohort / File(s)	Summary
Workflow Serialization .github/workflows/phpunit.yml	Re-serialized with Windows-style line endings (\r\n) and YAML formatting normalization; functional logic and workflow structure remain unchanged.
Version Specification Logic action.yml	Modified VERSION_SPEC determination: for workflow_dispatch events on branches, uses dev-${{ github.ref_name }}; otherwise uses ${{ github.ref_name }} or dev-${{ github.sha }} based on event type.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• SEC-1684: Fix/update matrix #26: Modifies VERSION_SPEC logic for composer package versions in action.yml with related conditional branching on GitHub event types.

Suggested reviewers

• jordandukart

Poem

🐰 A rabbit hops through GitHub's flow,
Where version specs now smartly know,
When to branch and when to sha,
The composer songs go la-di-da! 🎵

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title directly references the main change: adjusting VERSION_SPEC logic to handle dispatched workflows differently.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch adam-vessey-patch-1

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[adam-vessey]

Not sure why this got reformatted or whatever?

[adam-vessey]

Should probably be pointed back at the version that will be minted, prior to merging.

[github-actions]

Tag generated by PR: v1.5.3