Skip to content

refactor: consolidate dependencies to use fastlane-contracts directly#2

Merged
0x1NotMe merged 1 commit into
mainfrom
fix/remapping
May 29, 2025
Merged

refactor: consolidate dependencies to use fastlane-contracts directly#2
0x1NotMe merged 1 commit into
mainfrom
fix/remapping

Conversation

@0x1NotMe
Copy link
Copy Markdown
Contributor

@0x1NotMe 0x1NotMe commented May 27, 2025

Summary

This PR removes the separate fastlane-task-manager submodule and migrates all imports to use the consolidated fastlane-contracts library directly, eliminating dependency redundancy and simplifying the project structure.

Changes Made

  • Removed submodule: Deinitialized and removed lib/fastlane-task-manager
  • Updated remappings: Changed from @fastlane-task-manager/src= to @fastlane-contracts/=lib/fastlane-contracts/src/
  • Migrated imports across 7 files:
    • Task Manager: @fastlane-contracts/task-manager/interfaces/ITaskManager.sol
    • ShMonad: @fastlane-contracts/shmonad/interfaces/IShMonad.sol
    • Common: @fastlane-contracts/common/{IAddressHub,Directory}.sol
  • Cleaned up redundant code: Removed src/interfaces/shmonad/ folder (now using lib interfaces directly)
  • Updated tooling: VS Code settings updated to match new remappings

Testing Status

All builds and tests pass with new import structure

  • forge build: Successful compilation with only warnings
  • Tests: 24/28 passing (4 failing tests in BattleNadsMovementTest are pre-existing and unrelated to import changes)

- Remove fastlane-task-manager submodule (now part of fastlane-contracts)
- Update remappings to @fastlane-contracts/=lib/fastlane-contracts/src/
- Update all imports to use fastlane-contracts paths:
  - ITaskManager: @fastlane-contracts/task-manager/interfaces/
  - IShMonad: @fastlane-contracts/shmonad/interfaces/
  - Common interfaces: @fastlane-contracts/common/
- Remove redundant src/interfaces/shmonad/ folder
- Update VS Code settings to match new remappings
@0x1NotMe 0x1NotMe requested a review from thogard785 May 27, 2025 23:56
@0x1NotMe 0x1NotMe merged commit 24c913a into main May 29, 2025
0 of 2 checks passed
@0x1NotMe 0x1NotMe deleted the fix/remapping branch May 29, 2025 03:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant