Skip to content

Comments

Conditional config#57

Merged
jmnote merged 3 commits intojmnote:mainfrom
dummy-index:cleanup-and-cond
Jan 4, 2026
Merged

Conditional config#57
jmnote merged 3 commits intojmnote:mainfrom
dummy-index:cleanup-and-cond

Conversation

@dummy-index
Copy link
Contributor

Which issue this PR fixes / What this PR does / Why we need it

Added a feature to switch settings according to the revision ID. This allows settings to be changed without breaking the display of past articles. Note that this does not affect each past edit summaries or individually transcluded templates (which settings are applied are determined solely by the revision ID of the page).

Checklist

  • extension.json version bumped

@jmnote
Copy link
Owner

jmnote commented Jan 3, 2026

Thanks for the PR! I’m running an automated review with Codex first — I’ll follow up with a manual review shortly.

@codex review

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: bf8c6a07a5

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Copy link
Owner

@jmnote jmnote left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks~

@jmnote jmnote merged commit 26be020 into jmnote:main Jan 4, 2026
@dummy-index dummy-index deleted the cleanup-and-cond branch January 4, 2026 07:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants