Skip to content

Record why the timezone regression suite is not needed [ignore_release] - #206

Merged
snake14 merged 1 commit into
6.x-devfrom
timezone-regression-exempt
Sep 30, 2026
Merged

snake14 merged 1 commit into
6.x-devfrom
timezone-regression-exempt

Conversation

@snake14

@snake14 snake14 commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Description

Plugins CI now checks whether a plugin with date or site-timezone logic runs the timezone regression suite (matomo-org/plugin-ci-workflows#22). This plugin's date references cannot produce a wrong site day, so the Plugins CI call records the reason instead of adding a suite, which clears the new coverage warning: Generated visits go through the tracker, which invalidates the site-local day of each visit, so each generator also invalidating the UTC date is only an extra; the direct-to-database generator stores UTC timestamps and invalidates nothing.

No runtime change, so no version bump or changelog entry.

Issue No

Related to matomo-org/plugin-ci-workflows#22

Steps to Replicate the Issue

  1. Open this plugin's Plugins CI run on a pull request before this change.
  2. Expected: no timezone regression coverage warning.
  3. Actual: the timezone safety job warns that the plugin has no timezone regression suite.

Checklist

  • [✔] Tested locally or on demo2/demo3?
  • [NA] New test case added/updated?
  • [NA] Are all newly added texts included via translation?
  • [NA] Are text sanitized properly? (Eg use of v-text v/s v-html for vue)
  • [NA] Version bumped?
  • [✔] I have understood, reviewed, and tested all AI outputs before use
  • [✔] All AI instructions respect security, IP, and privacy rules
  • [NA] Documentation updated?

@snake14
snake14 enabled auto-merge (squash) September 30, 2026 22:51
@snake14
snake14 requested a review from a team September 30, 2026 22:51
@snake14
snake14 merged commit 949f8c5 into 6.x-dev Sep 30, 2026
32 of 34 checks passed
@snake14
snake14 deleted the timezone-regression-exempt branch September 30, 2026 23:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants