Skip to content

Refactor Out Deprecations - #514

Open
martindale wants to merge 10 commits into
honkit:masterfrom
martindale:feature/deprecations
Open

martindale wants to merge 10 commits into
honkit:masterfrom
martindale:feature/deprecations

Conversation

@martindale

Copy link
Copy Markdown
Contributor

Container for work on eliminating all deprecations. Initial work adds two new commands, report:install and a CI-safe equivalent report:install-ci as well as a reports/ directory to contain the reference artifact.

@martindale
martindale force-pushed the feature/deprecations branch from 9a78d03 to 874a52b Compare May 17, 2026 06:28
Replace cpr/cp directory copies with Node fs helpers, swap tiny-lr for
livereload (no raw-body chain), and require Node >=20. Refresh lockfile
and reports/install.log after report:install.
@martindale

Copy link
Copy Markdown
Contributor Author

I'm flipping this over to ready for review, but I'm still not entirely happy with it. Seeking comments.

@martindale
martindale marked this pull request as ready for review May 19, 2026 02:22
devin-ai-integration[bot]

This comment was marked as resolved.

Comment thread packages/@honkit/html/package.json Outdated
Comment thread package.json Outdated
devin-ai-integration[bot]

This comment was marked as resolved.

@martindale martindale changed the title WIP: Refactor Out Deprecations Refactor Out Deprecations May 22, 2026
@martindale

Copy link
Copy Markdown
Contributor Author

I've pushed a few more commits and am pretty happy with this PR now. Removed WIP from the title.

@azu azu added the Type: Refactoring A code change that neither fixes a bug nor adds a feature label May 25, 2026
Comment thread reports/install.log

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Does this file need to be committed?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

No, but I like having it to serve as an artifact — it's good to keep track of the dependency changes over time. Shall I remove it?

devin-ai-integration[bot]

This comment was marked as resolved.

This branch has not been deployed

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

Labels

Type: Refactoring A code change that neither fixes a bug nor adds a feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants