Skip to content

Fix ADR-004 description in ADR-002 intro note#262

Merged
shorodilov merged 2 commits into
docs/clean-upfrom
copilot/sub-pr-260
Jan 31, 2026
Merged

Fix ADR-004 description in ADR-002 intro note#262
shorodilov merged 2 commits into
docs/clean-upfrom
copilot/sub-pr-260

Conversation

Copilot AI commented Jan 31, 2026

Copy link
Copy Markdown

ADR-002's intro note described ADR-004 as "impress.js submodule → npm dependency", but ADR-004 was revised to focus solely on removing the submodule. Presentation separation is now captured in ADR-006.

Changes:

  • Updated ADR-004 description to "removing impress.js submodule"
  • Added ADR-006 reference for presentation repository separation
  • Removed "three interdependent" count (now four ADRs total)
 > - **ADR-002** (this document): Static Site Generator Replacement (Sphinx → MkDocs)
 > - **ADR-003**: Repository File Structure (locale-based content organization)
-> - **ADR-004**: Presentation Framework Handling (impress.js submodule → npm dependency)
+> - **ADR-004**: Presentation Framework Handling (removing impress.js submodule)
+> - **ADR-006**: Presentation Content Repository Separation

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: shorodilov <22391544+shorodilov@users.noreply.github.com>
Copilot AI changed the title [WIP] WIP address feedback on documentation clean-up Fix ADR-004 description in ADR-002 intro note Jan 31, 2026
Copilot AI requested a review from shorodilov January 31, 2026 01:22
@shorodilov shorodilov marked this pull request as ready for review January 31, 2026 01:24
@shorodilov shorodilov merged commit eadbf81 into docs/clean-up Jan 31, 2026
@shorodilov shorodilov deleted the copilot/sub-pr-260 branch January 31, 2026 01:28
shorodilov added a commit that referenced this pull request Jan 31, 2026
* Initial plan

* fix(docs): update ADR-002 to reflect revised ADR-004 scope

Co-authored-by: shorodilov <22391544+shorodilov@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: shorodilov <22391544+shorodilov@users.noreply.github.com>
shorodilov added a commit that referenced this pull request Feb 1, 2026
* Initial plan

* fix(docs): update ADR-002 to reflect revised ADR-004 scope

Co-authored-by: shorodilov <22391544+shorodilov@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: shorodilov <22391544+shorodilov@users.noreply.github.com>
shorodilov added a commit that referenced this pull request Feb 2, 2026
* Initial plan

* fix(docs): update ADR-002 to reflect revised ADR-004 scope

Co-authored-by: shorodilov <22391544+shorodilov@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: shorodilov <22391544+shorodilov@users.noreply.github.com>
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