fix: correct relay package imports - #1
Conversation
|
This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation. |
There was a problem hiding this comment.
Pull Request Overview
This PR fixes import paths across the relay package to correctly reference local packages under the src/ directory. The changes ensure that all internal package imports are updated to include the proper src/ path prefix, and updates the protobuf go_package option accordingly.
- Updated import paths from
github.com/qubic/wargame-relay/togithub.com/qubic/wargame-relay/src/for internal packages - Modified protobuf go_package option to match the new directory structure
- Standardized import paths across main.go, API handlers, and scheduler components
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| relay/src/worker/worker.proto | Updated go_package option to include src/ path prefix |
| relay/src/scheduler/scheduler.go | Fixed blockchain package import to use src/ path |
| relay/src/main.go | Updated all internal package imports to use src/ path prefix |
| relay/src/api/handler.go | Fixed scheduler and worker package imports to use src/ path |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Summary
Testing
make test(fails: missing go.sum entries for external dependencies)https://chatgpt.com/codex/tasks/task_e_68a8312789c4832384137d6d56abe7e4