Skip to content

docs: output markdown files #1135

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 11 commits into from
Sep 8, 2025
Merged

docs: output markdown files #1135

merged 11 commits into from
Sep 8, 2025

Conversation

ilbertt
Copy link
Member

@ilbertt ilbertt commented Sep 7, 2025 •
edited
Loading

Description

Switches to generating markdown assets for docs.

Blocked by:

Checklist:

  • My changes follow the guidelines in CONTRIBUTING.md.
  • The title of this PR complies with Conventional Commits.
  • I have edited the CHANGELOG accordingly.
  • I have made corresponding changes to the documentation.

@ilbertt ilbertt changed the title Luca/markdown-docs docs: output markdown files Sep 7, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Sep 7, 2025 •
edited
Loading

size-limit report 📦

Path Size
@dfinity/agent 50.46 KB (0%)
@dfinity/assets 55.52 KB (0%)
@dfinity/auth-client 21.15 KB (0%)
@dfinity/candid 13.32 KB (0%)
@icp-sdk/core 91 B (0%)
@dfinity/identity 20.59 KB (0%)
@dfinity/identity-secp256k1 33.88 KB (0%)
@dfinity/principal 4.44 KB (0%)

@ilbertt ilbertt marked this pull request as ready for review September 8, 2025 13:07
@ilbertt ilbertt requested a review from a team as a code owner September 8, 2025 13:07
@ilbertt ilbertt merged commit e45f944 into main Sep 8, 2025
28 checks passed
@ilbertt ilbertt deleted the luca/markdown-docs branch September 8, 2025 14:56
@ilbertt ilbertt mentioned this pull request Sep 8, 2025
Merged
ilbertt added a commit that referenced this pull request Sep 9, 2025
@ilbertt
docs: fix cleaning dir before copying assets (#1137)
5c71b0f 
Tested with a temporary action that has now been removed. Useful to test changes made in #1135 as well.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lwshang lwshang lwshang approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ilbertt @lwshang