Skip to content

Integrate schema generator #25

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

Open
LukeHagar wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Integrate schema generator #25

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
7e5aa90
Saving progress on paths and methods
LukeHagar Jul 4, 2024
0b3842c
added method page
LukeHagar Jul 5, 2024
825594a
pnpm i
LukeHagar Jul 5, 2024
98630d2
removed lock
LukeHagar Jul 5, 2024
6f9b979
fix variable usage
LukeHagar Jul 5, 2024
02dc997
Marked pages not yet implemented
LukeHagar Jul 5, 2024
bf0b0da
Saving endpoint progress
LukeHagar Jul 6, 2024
2d6c042
Added plausible, adjusted label
LukeHagar Jul 10, 2024
6bd170a
updated-deps
LukeHagar Jul 10, 2024
a6e8650
manually set pnpm version
LukeHagar Jul 10, 2024
146b490
updated script tag
LukeHagar Oct 25, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 9 additions & 7 deletions package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,17 +14,17 @@
"devDependencies": {
"@skeletonlabs/skeleton": "2.10.0",
"@skeletonlabs/tw-plugin": "0.4.0",
"@sveltejs/adapter-auto": "^3.2.2",
"@sveltejs/adapter-node": "^5.2.0",
"@sveltejs/kit": "^2.5.18",
"@sveltejs/vite-plugin-svelte": "^3.1.1",
"@tailwindcss/forms": "0.5.7",
"@tailwindcss/typography": "0.5.13",
"@types/eslint": "^8.56.10",
"@types/node": "20.12.12",
"@typescript-eslint/eslint-plugin": "^7.15.0",
"@typescript-eslint/parser": "^7.15.0",
"@typescript-eslint/eslint-plugin": "^7.16.0",
"@typescript-eslint/parser": "^7.16.0",
"autoprefixer": "10.4.19",
"dexie": "^4.0.7",
"dexie": "^4.0.8",
"eslint": "^8.57.0",
"eslint-config-prettier": "^9.1.0",
"eslint-plugin-svelte": "^2.41.0",
Expand All @@ -50,6 +50,8 @@
"svelte-persisted-store": "^0.9.4"
},
"engines": {
"node": ">20.0.0"
}
}
"node": ">20.0.0",
"pnpm": "8"
Comment on lines +53 to +54
Copy link

@coderabbitai coderabbitai bot Oct 25, 2024

Choose a reason for hiding this comment

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

💡 Codebase verification

Node.js version requirement needs documentation and CI setup

The project requires Node.js >20.0.0, but this requirement is not documented in README.md and there are no CI configuration files detected. Several dependencies in pnpm-lock.yaml already support Node.js 20, indicating compatibility.

  • Add Node.js and pnpm version requirements to README.md
  • Set up CI/CD configuration with the specified Node.js version
  • Consider adding a version check script in package.json
🔗 Analysis chain

Verify Node.js version compatibility.

The engine requirements specify Node.js >20.0.0 and pnpm 8, which are relatively recent versions. This might impact development and deployment environments.

Ensure all development and CI/CD environments meet these requirements. Consider:

  • Updating CI/CD pipeline configurations
  • Documenting the requirements in README
  • Adding version checks in development scripts

Let me check the CI/CD configuration files to get a complete picture.

Let me check for CI configuration files specifically to complete the verification.

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Check if README mentions Node.js version requirement
rg -i "node(\s+)?([>=])?20" README.md || echo "Node.js version requirement not documented in README.md"

Length of output: 160

Script:

#!/bin/bash
# Check CI/CD configuration files for Node.js version
fd -e yml -e yaml | xargs rg -i "node(\s+)?([>=])?20|node-version"

# Check for any development scripts that might contain version checks
fd -e sh -e js package.json | xargs rg -i "node(\s+)?version|engine"

Length of output: 12679

Script:

#!/bin/bash
# Check for CI configuration files in common locations
fd -e yml -e yaml -p '.github/workflows|.gitlab-ci|.circleci'

Length of output: 63

},
"packageManager": "pnpm@8.15.8+sha512.d1a029e1a447ad90bc96cd58b0fad486d2993d531856396f7babf2d83eb1823bb83c5a3d0fc18f675b2d10321d49eb161fece36fe8134aa5823ecd215feed392"
}
Loading