Skip to content

Conversation

@Leftbower
Copy link
Contributor

Description

If docs are correct,then the Commandbox managed modules folder should be in the lib folder (along with Coldbox and Testbox). Ths requires a few changes:

  1. Change the folder structure to point there
  2. Add a installPathConventions directive in box.json to point new module installs there
  3. Address the discrepency in the /modules cfmapping between the Applicationn.cfc files in the Public folder and the tests folder.

Please note that all PRs must have tests attached to them

IMPORTANT: Please review the CONTRIBUTING.md file for detailed contributing guidelines.

Issues

#13

Type of change

Please delete options that are not relevant.

  • Bug Fix
  • Improvement
  • New Feature
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist

  • My code follows the style guidelines of this project cfformat
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

@lmajano lmajano merged commit 340187e into coldbox-templates:development Nov 10, 2025
5 checks passed
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.

2 participants