-
Notifications
You must be signed in to change notification settings - Fork 45
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
fix: submodule #272
fix: submodule #272
Changes from 10 commits
55ed064
52743fa
b85f225
5314a67
5cec278
09eaeac
49afac9
922fb99
e4368f5
16dbaaa
e3000bc
b7fe164
46e6fc3
41a0bf0
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,57 @@ | ||
name: Release | ||
|
||
on: | ||
workflow_dispatch: | ||
inputs: | ||
release_type: | ||
description: "Type of release (prerelease, prepatch, patch, minor, preminor, major)" | ||
required: true | ||
default: "patch" | ||
|
||
jobs: | ||
release: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
with: | ||
fetch-depth: 0 | ||
|
||
- uses: pnpm/action-setup@v3 | ||
with: | ||
version: 8 | ||
|
||
- name: Configure Git | ||
run: | | ||
git config user.name "${{ github.actor }}" | ||
git config user.email "${{ github.actor }}@users.noreply.github.com" | ||
- name: "Setup npm for npmjs" | ||
run: | | ||
npm config set registry https://registry.npmjs.org/ | ||
echo "//registry.npmjs.org/:_authToken=${{ secrets.NPM_TOKEN }}" > ~/.npmrc | ||
- name: Install Protobuf Compiler | ||
run: sudo apt-get install -y protobuf-compiler | ||
|
||
- name: Install dependencies | ||
run: pnpm install | ||
|
||
- name: Build packages | ||
run: pnpm run build | ||
|
||
- name: Tag and Publish Packages | ||
run: | | ||
npx lerna version ${{ github.event.inputs.release_type }} --conventional-commits --yes --no-private --force-publish | ||
npx lerna publish from-git --yes --dist-tag ${{ github.event.inputs.release_type == 'preminor' && 'next' || 'latest' }} | ||
- name: Generate Changelog | ||
run: | | ||
npx lerna-changelog > CHANGELOG.md | ||
- name: Commit and Push Changelog | ||
run: | | ||
git add CHANGELOG.md | ||
git commit -m "chore(release): update changelog [skip ci]" | ||
git push origin HEAD:${{ github.ref }} | ||
env: | ||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} |
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,3 @@ | ||
[submodule "dojo-starter"] | ||
path = examples/dojo-starter | ||
path = worlds/dojo-starter | ||
url = https://github.com/dojoengine/dojo-starter |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -17,29 +17,29 @@ | |
"@dojoengine/state": "workspace:*", | ||
"@dojoengine/torii-client": "workspace:*", | ||
"@dojoengine/utils": "workspace:*", | ||
"@latticexyz/react": "^2.0.12", | ||
"@latticexyz/utils": "^2.0.12", | ||
"@latticexyz/react": "^2.2.8", | ||
"@latticexyz/utils": "^2.2.8", | ||
"ethers": "^5.7.2", | ||
"mobx": "^6.9.0", | ||
"mobx": "^6.13.2", | ||
"proxy-deep": "^3.1.1", | ||
"react": "^18.2.0", | ||
"react-dom": "^18.2.0", | ||
"react": "^18.3.1", | ||
"react-dom": "^18.3.1", | ||
"rxjs": "^7.8.1", | ||
"starknet": "6.11.0", | ||
"vite-plugin-top-level-await": "^1.3.1", | ||
"vite-plugin-wasm": "^3.2.2" | ||
"vite-plugin-top-level-await": "^1.4.4", | ||
"vite-plugin-wasm": "^3.3.0" | ||
}, | ||
"devDependencies": { | ||
"@types/node": "^20.4.8", | ||
"@types/react": "^18.2.33", | ||
"@types/react-dom": "^18.0.11", | ||
"@typescript-eslint/eslint-plugin": "^5.59.0", | ||
"@typescript-eslint/parser": "^5.59.0", | ||
"@vitejs/plugin-react": "^4.0.0", | ||
"eslint": "^8.38.0", | ||
"eslint-plugin-react-hooks": "^4.6.0", | ||
"eslint-plugin-react-refresh": "^0.3.4", | ||
"typescript": "^5.5.4", | ||
"vite": "^4.3.9" | ||
"@types/node": "^20.16.6", | ||
"@types/react": "^18.3.9", | ||
"@types/react-dom": "^18.3.0", | ||
"@typescript-eslint/eslint-plugin": "^5.62.0", | ||
"@typescript-eslint/parser": "^5.62.0", | ||
"@vitejs/plugin-react": "^4.3.1", | ||
"eslint": "^8.57.1", | ||
"eslint-plugin-react-hooks": "^4.6.2", | ||
"eslint-plugin-react-refresh": "^0.3.5", | ||
"typescript": "^5.6.2", | ||
"vite": "^4.5.5" | ||
Comment on lines
+33
to
+43
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. 💡 Codebase verification Missing Essential devDependencies Detected The current
Analysis chainVerify development environment compatibility with updated devDependencies. The devDependencies have been updated to newer versions. These updates may affect your development environment and build process. Key changes include:
To ensure these updates don't negatively impact your development workflow, run the following script: After running these checks, manually verify that your development workflow (hot reloading, build process, etc.) still functions as expected. Scripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Description: Verify compatibility of updated devDependencies
# Test: Check if TypeScript compilation works without errors
npx tsc --noEmit
# Test: Run ESLint to check for any new linting issues
npx eslint .
# Test: Verify Vite build process
npm run build
echo "Please review the output of these commands for any unexpected errors or warnings."
Length of output: 436 Script: #!/bin/bash
# Description: Extract and display devDependencies from package.json
if command -v jq >/dev/null 2>&1; then
jq '.devDependencies' package.json
else
echo "jq is not installed. Please install jq to parse JSON files."
fi
Length of output: 241 |
||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Codebase verification
Action Required: Update remaining references to the old submodule path.
The following files still reference
examples/dojo-starter
and should be updated toworlds/dojo-starter
:readme.md
cd examples/dojo-starter
cd examples/dojo-starter
examples/example-nodejs-bot/tsconfig.json
"exclude": ["examples/dojo-starter/target/dev/manifest.json"]
Please ensure these paths are updated to reflect the new submodule location to maintain consistency across the project.
Analysis chain
LGTM! Verify consistency across the project.
The change to the submodule path from
examples/dojo-starter
toworlds/dojo-starter
looks good. This appears to be a directory restructuring, which aligns with the PR title "fix: submodule".To ensure consistency across the project, please run the following script to check for any references to the old path:
This script will help identify any potential inconsistencies or references that might need updating due to this path change.
LGTM! Verify no residual references to the old submodule path.
To ensure that there are no remaining references to
examples/dojo-starter
outside of.gitmodules
, please run the following script:Scripts executed
The following scripts were executed for the analysis:
Script:
Length of output: 587
Script:
Length of output: 394