Skip to content

chore: monorepo rename#22

Merged
hackmajoris merged 5 commits intomainfrom
chore/monorepo-rename
Dec 21, 2025
Merged

chore: monorepo rename#22
hackmajoris merged 5 commits intomainfrom
chore/monorepo-rename

Conversation

@hackmajoris
Copy link
Copy Markdown
Owner

@hackmajoris hackmajoris commented Dec 21, 2025

Changes

  • rename the monorepo from glad to gald-stack
  • rename the cmd/app to cmd/glad
  • split Taskfile into a global and app level specific.

Copy link
Copy Markdown
Contributor

@amazon-q-developer amazon-q-developer bot left a comment

Choose a reason for hiding this comment

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

Summary

This PR implements a monorepo rename from "app" to "glad", which is a significant structural change. However, there are several critical issues that will prevent the project from building and deploying correctly:

Critical Issues Found:

  1. 🚨 Configuration Path Mismatch: The new cmd/glad/Taskfile.yml references deployments/glad but this directory doesn't exist - it should be deployments/app

  2. 🚨 CI/CD Broken: The GitHub Actions workflow references task build:lambda which no longer exists in the root Taskfile.yml

  3. 🚨 Documentation Inconsistency: README.md shows deployments/glad/ in the project structure but the actual directory is still deployments/app/

  4. ⚠️ Version Mismatch: CI workflow uses Go 1.21 but go.mod specifies Go 1.24.0

Positive Changes:

  • ✅ Import paths correctly updated throughout Go files
  • ✅ Monorepo structure with includes properly implemented
  • ✅ Docker build arguments added for flexibility
  • ✅ Task organization improved with app-specific tasks

Recommendation:

Do not merge until the critical path mismatches are resolved. The deployment directory structure needs to be either:

  1. Renamed from deployments/app/ to deployments/glad/, OR
  2. All references updated to use deployments/app/

The CI workflow also needs to be updated to use the correct task commands from the new monorepo structure.


You can now have the agent implement changes and create commits directly on your pull request's source branch. Simply comment with /q followed by your request in natural language to ask the agent to make changes.

@hackmajoris hackmajoris merged commit f35d7ae into main Dec 21, 2025
2 checks passed
@hackmajoris hackmajoris deleted the chore/monorepo-rename branch December 21, 2025 20:14
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