-
Notifications
You must be signed in to change notification settings - Fork 32
feat(sdk-docs): add documentation configuration for SDK monorepo #522
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.
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
Changes from 1 commit
e92e6b0
842eeb8
5a12ea8
f561cd5
435eba3
0c59957
64507cb
b98b4c3
bacdcac
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,10 @@ | ||
| **/* | ||
| !_category_.json | ||
| !_category_.yml | ||
| !provider-adapters/ | ||
| !provider-adapters/_category_.json | ||
| !core-utilities/ | ||
| !core-utilities/_category_.json | ||
| !protocol-components/ | ||
| !protocol-components/_category_.json | ||
| !.gitignore | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,8 @@ | ||
| { | ||
| "label": "Core Utilities", | ||
| "position": 3, | ||
| "link": { | ||
| "type": "generated-index", | ||
| "description": "Core utility modules for the Cow Protocol SDK" | ||
| } | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,8 @@ | ||
| { | ||
| "label": "Protocol Components", | ||
| "position": 4, | ||
| "link": { | ||
| "type": "generated-index", | ||
| "description": "Protocol-specific components and modules" | ||
| } | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,8 @@ | ||
| { | ||
| "label": "Provider Adapters", | ||
| "position": 2, | ||
| "link": { | ||
| "type": "generated-index", | ||
| "description": "Provider adapters for different Ethereum libraries" | ||
| } | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,11 +6,9 @@ | |
| "docusaurus": "docusaurus", | ||
| "start": "docusaurus start", | ||
| "git:clone:cowsdk": "cd external && rm -rf cow-sdk && git clone --depth=1 --branch=main https://github.com/cowprotocol/cow-sdk cow-sdk && rm -rf cow-sdk/.git", | ||
| "git:clone:appdata": "cd external && rm -rf app-data && git clone --depth=1 --branch=main https://github.com/cowprotocol/app-data app-data && rm -rf app-data/.git", | ||
| "build:external:cowsdk": "yarn git:clone:cowsdk && cd external/cow-sdk && yarn install --frozen --ignore-scripts && yarn codegen", | ||
| "build:external:appdata": "yarn git:clone:appdata && cd external/app-data && yarn install --frozen --ignore-scripts && yarn compile", | ||
| "build:external": "yarn build:external:cowsdk && yarn build:external:appdata", | ||
| "build": "yarn build:external && docusaurus build", | ||
| "build:external:cowsdk": "cd external/cow-sdk && pnpm install && pnpm run build", | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Verification agent 🧩 Analysis chainEnsure pnpm is available in the build environment The build script has switched from 🏁 Script executed: #!/bin/bash
# Check if pnpm is available
which pnpm || echo "pnpm not found"
# Check if there's any CI configuration that might need updating
fd -e yml -e yaml . .github/workflows/ | xargs grep -l "yarn" | head -5Length of output: 219 Add pnpm installation to CI and local setup Our check showed that • CI/CD workflows (e.g. in .github/workflows/*.yml): These changes will guarantee 🤖 Prompt for AI Agents
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Good point. @jeffersonBastos if pnpm is now a requirement for the build, let's add it to the dependencies
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Nice catch!
|
||
| "build:external": "yarn build:external:cowsdk", | ||
| "build": "yarn build:external && NODE_OPTIONS=\"--max-old-space-size=8192\" docusaurus build", | ||
| "swizzle": "docusaurus swizzle", | ||
| "deploy": "docusaurus deploy", | ||
| "clear": "docusaurus clear", | ||
|
|
@@ -73,4 +71,4 @@ | |
| "_comment": "https://github.com/swagger-api/swagger-ui/releases/tag/v5.18.0", | ||
| "enabled": false | ||
| } | ||
| } | ||
| } | ||
Uh oh!
There was an error while loading. Please reload this page.