Make test-with-coverage.yml workflow run in push events to master branch

[jrosental]

Summary by CodeRabbit

• Chores
  • Updated test automation workflow configuration to enhance code coverage monitoring and testing frequency.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

The pull request adds a push trigger to the GitHub Actions workflow for the master branch while ignoring tagged pushes. The workflow now executes on both pull requests and pushes to master, expanding its event-driven execution beyond the existing pull request trigger.

Changes

Cohort / File(s)	Summary
Workflow trigger configuration .github/workflows/test-with-coverage.yml	Adds a push event trigger for the master branch with tags-ignore: ['*.*'] to prevent execution on tagged releases, while maintaining existing pull_request trigger

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• split test-with-coverage.yml workflow #505: Previous modifications to the same workflow file's trigger configuration and execution flow.

Suggested reviewers

• alexeykazakov

Poem

🐰 The workflow now dances both ways,
On pushes and PRs through the days,
Tags take a bow and gracefully pass,
While master's tests run full blast!
CI/CD springs forth with glee,
A swift review is all you need, you'll see! ✨

🚥 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 accurately and specifically describes the main change: adding push event triggers to the workflow for the master branch.
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.}

---

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.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In @.github/workflows/test-with-coverage.yml:
- Around line 5-9: The tags-ignore pattern currently set to '*.*' only matches
tags with a dot; update the workflow key tags-ignore under the push trigger to
use '*' instead of '*.*' so all tag pushes are ignored (edit the YAML entry
tags-ignore: to tags-ignore: - '*' preserving indentation and list format).

[coderabbitai]

⚠️ Potential issue | 🟡 Minor

`tags-ignore: '.' won’t ignore all tag pushes.

This only matches tags containing a dot (e.g., v1.2). If the goal is to ignore all tags, use '*'.

✅ Suggested fix

     tags-ignore:
-      - '*.*'
+      - '*'

📝 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

	push:
	branches:
	- master
	tags-ignore:
	- '*.*'
	push:
	branches:
	- master
	tags-ignore:
	- '*'

🤖 Prompt for AI Agents

In @.github/workflows/test-with-coverage.yml around lines 5 - 9, The tags-ignore
pattern currently set to '*.*' only matches tags with a dot; update the workflow
key tags-ignore under the push trigger to use '*' instead of '*.*' so all tag
pushes are ignored (edit the YAML entry tags-ignore: to tags-ignore: - '*'
preserving indentation and list format).