Skip to content
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

Merged
merged 14 commits into from
Sep 25, 2024
2 changes: 1 addition & 1 deletion .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ jobs:
- run: curl -L https://install.dojoengine.org | bash
- run: /home/runner/.config/.dojo/bin/dojoup -v v1.0.0-alpha.6
- run: |
cd examples/dojo/dojo-starter
cd worlds/dojo-starter
/home/runner/.config/.dojo/bin/sozo build
/home/runner/.config/.dojo/bin/sozo test

Expand Down
2 changes: 1 addition & 1 deletion .gitmodules
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
Copy link

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 to worlds/dojo-starter:

  • readme.md
    • Line with cd examples/dojo-starter
    • Another line with 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 to worlds/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:

#!/bin/bash
# Description: Check for references to the old submodule path

# Search for references to the old path
echo "Searching for references to 'examples/dojo-starter':"
rg --type-not gitmodules "examples/dojo-starter"

# Check if the new directory structure exists
echo "Checking if the new directory structure exists:"
if [ -d "worlds/dojo-starter" ]; then
    echo "Directory 'worlds/dojo-starter' exists."
else
    echo "Warning: Directory 'worlds/dojo-starter' does not exist."
fi

# Verify submodule status
echo "Verifying submodule status:"
git submodule status

Length of output: 587


Script:

#!/bin/bash
# Description: Check for references to the old submodule path excluding .gitmodules

echo "Searching for references to 'examples/dojo-starter' excluding .gitmodules:"
rg "examples/dojo-starter" --glob '!**/.gitmodules'

Length of output: 394

url = https://github.com/dojoengine/dojo-starter
Loading