Skipped Tinybird tests job on forks

[9larsons]

no ref

This task requires a secret that's not available on forks, and we should skip it like we do other jobs.

[coderabbitai]

Walkthrough

The conditional logic for a Tinybird workflow step in the CI pipeline was modified. Previously, the step ran only when the repository matched a specific value. The new logic allows the step to execute for all non-pull-request events, while for pull-request events, it runs only when the PR's head repository matches the current repository. This change adjusts when the traffic analytics infrastructure workflow is triggered during the CI process.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: skipping the Tinybird tests job on forks, which is the core issue being fixed.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check	✅ Passed	The pull request description briefly mentions the issue and solution, relating directly to the changeset which fixes a Tinybird CI dispatch step conditional for fork PRs.

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

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix-tinybird-ci-fork-check

---

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

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.46%. Comparing base (17b5b5a) to head (adff9c7).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #27344   +/-   ##
=======================================
  Coverage   73.46%   73.46%           
=======================================
  Files        1545     1545           
  Lines      123724   123724           
  Branches    14970    14970           
=======================================
+ Hits        90893    90894    +1     
- Misses      31808    31828   +20     
+ Partials     1023     1002   -21     

Flag	Coverage Δ
admin-tests	54.36% <ø> (+<0.01%)	⬆️
e2e-tests	73.46% <ø> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.