Skip to content

Conversation

@d3xter666
Copy link
Member

@d3xter666 d3xter666 commented Nov 18, 2025

The isolated documentation package does not have its own package-lock.json, so the documentation publishing action fails on npm ci command.

@d3xter666 d3xter666 requested a review from a team November 18, 2025 15:43
@coveralls
Copy link

Coverage Status

coverage: 94.67% (-0.1%) from 94.772%
when pulling f67fd0d on fix-documentation-workflow
into 5f6f20c on main.

2 similar comments
@coveralls
Copy link

Coverage Status

coverage: 94.67% (-0.1%) from 94.772%
when pulling f67fd0d on fix-documentation-workflow
into 5f6f20c on main.

@coveralls
Copy link

Coverage Status

coverage: 94.67% (-0.1%) from 94.772%
when pulling f67fd0d on fix-documentation-workflow
into 5f6f20c on main.

@coveralls
Copy link

coveralls commented Nov 18, 2025

Coverage Status

coverage: 94.624% (-0.03%) from 94.657%
when pulling 7100e68 on fix-documentation-workflow
into 1958d40 on main.

@d3xter666 d3xter666 marked this pull request as draft November 18, 2025 17:02
@d3xter666 d3xter666 marked this pull request as ready for review November 19, 2025 07:30
@d3xter666 d3xter666 changed the title fix: Run npm install for API documentation of the isolated package fix: Documentation build with NPM depednecnies Nov 19, 2025
@d3xter666 d3xter666 requested a review from a team November 19, 2025 07:33
@d3xter666 d3xter666 force-pushed the fix-documentation-workflow branch from 1768021 to 499e5cf Compare November 21, 2025 08:00
@RandomByte RandomByte changed the title fix: Documentation build with NPM depednecnies fix: Documentation build with NPM depednencies Nov 21, 2025
@RandomByte RandomByte changed the title fix: Documentation build with NPM depednencies fix: Documentation build with NPM dependencies Nov 21, 2025
@RandomByte
Copy link
Member

LGTM, giving @matz3 a chance to review as well

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.

5 participants