Skip to content

Comments

Separate screenshot script changes from Arabic translation PR#185

Merged
pethers merged 3 commits intotssfrom
copilot/sub-pr-181
Feb 15, 2026
Merged

Separate screenshot script changes from Arabic translation PR#185
pethers merged 3 commits intotssfrom
copilot/sub-pr-181

Conversation

Copy link
Contributor

Copilot AI commented Feb 15, 2026

PR #181's code review identified scope creep: commit c06c5e6 mixed unrelated changes (screenshot script, sample file cleanup, JSON-LD fixes) with Arabic translations.

Changes:

  • Revert scripts/take-screenshot.js to reference sample-economist-style.html (was changed to 2026-02-13-evening-analysis-en.html)
  • Restore deleted sample files (news/sample-economist-style.html, news/2026-02-14-sample-no-embedded-css-en.html)
  • Revert scripts/generate-news-indexes.js and index file changes (EN/SV)

Result:

-  const filePath = join(__dirname, '..', 'news', '2026-02-13-evening-analysis-en.html');
+  const filePath = join(__dirname, '..', 'news', 'sample-economist-style.html');

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Update Arabic translations for evening analysis article Separate screenshot script changes from Arabic translation PR Feb 15, 2026
Copilot AI requested a review from pethers February 15, 2026 10:54
@pethers pethers marked this pull request as ready for review February 15, 2026 10:59
@pethers pethers merged commit b7b759c into tss Feb 15, 2026
1 check passed
@pethers pethers deleted the copilot/sub-pr-181 branch February 15, 2026 10:59
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.

2 participants