Skip to content

Revert "feat: use relative-time element for PR timestamp localization"#178

Merged
Paveltarno merged 1 commit intomainfrom
revert-176-fix/issue-175-pr-timestamp-localization
Feb 2, 2026
Merged

Revert "feat: use relative-time element for PR timestamp localization"#178
Paveltarno merged 1 commit intomainfrom
revert-176-fix/issue-175-pr-timestamp-localization

Conversation

@Paveltarno
Copy link
Contributor

@Paveltarno Paveltarno commented Feb 2, 2026

Note

Description

This PR reverts the recent change that introduced GitHub's <relative-time> web component for timestamp localization in auto-generated PR descriptions. The revert restores the original static UTC timestamp format (YYYY-MM-DD HH:MM UTC) instead of the ISO 8601 format with dynamic relative time display.

Related Issue

Reverts #176

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactoring (no functional changes)
  • Other (please describe): Revert of a previous feature

Changes Made

  • Reverted .github/workflows/pr-description.yml line 81
  • Changed timestamp instruction back from ISO 8601 format with <relative-time> element to static UTC format
  • Restored original timestamp placeholder: [current UTC timestamp in format: YYYY-MM-DD HH:MM UTC]

Testing

  • I have tested these changes locally
  • I have added/updated tests as needed
  • All tests pass (npm test)

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (if applicable)
  • My changes generate no new warnings
  • I have updated AGENTS.md if I made architectural changes

Additional Notes

This revert undoes the timestamp localization feature that was introduced in PR #176. The change is minimal (1 line modified) and restores the workflow to its previous stable state.


🤖 Generated by Claude | 2026-02-02 12:47 UTC

@Paveltarno Paveltarno merged commit 19b975d into main Feb 2, 2026
6 checks passed
@Paveltarno Paveltarno deleted the revert-176-fix/issue-175-pr-timestamp-localization branch February 2, 2026 12:44
@github-actions
Copy link
Contributor

github-actions bot commented Feb 2, 2026

🚀 Package Preview Available!


Install this PR's preview build with npm:

npm i @base44-preview/cli@0.0.26-pr.178.90895a6

Prefer not to change any import paths? Install using npm alias so your code still imports base44:

npm i "base44@npm:@base44-preview/cli@0.0.26-pr.178.90895a6"

Or add it to your package.json dependencies:

{
  "dependencies": {
    "base44": "npm:@base44-preview/cli@0.0.26-pr.178.90895a6"
  }
}

Preview published to npm registry — try new features instantly!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant