Skip to content

Conversation

@mrubens
Copy link
Collaborator

@mrubens mrubens commented Jun 16, 2025

Important

Fix typo in MdmService.ts and tests to use mdm.json instead of mcp.json.

  • Behavior:
    • Fix typo in getMdmConfigPath() in MdmService.ts to use mdm.json instead of mcp.json for configuration file names.
    • Update comments in MdmService.ts to reflect the correct file names for Windows, macOS, and Linux.
  • Tests:
    • Update MdmService.spec.ts to check for mdm.json and mdm.dev.json instead of mcp.json and mcp.dev.json in various test cases.

This description was created by Ellipsis for a7cf60b. You can customize this summary. It will automatically update as commits are pushed.

@mrubens mrubens requested review from cte and jr as code owners June 16, 2025 23:10
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Jun 16, 2025
@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Jun 16, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jun 16, 2025
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Jun 16, 2025
@mrubens mrubens merged commit c10fbbc into main Jun 16, 2025
12 checks passed
@mrubens mrubens deleted the fix_typo branch June 16, 2025 23:20
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jun 16, 2025
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Jun 16, 2025
shivangag pushed a commit to IdenWorks/Roo-Code that referenced this pull request Jun 17, 2025
* Fix typo in mdm.json

* Update src/services/mdm/MdmService.ts

Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>

---------

Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
valekseev pushed a commit to valekseev/Roo-Code that referenced this pull request Jun 18, 2025
* Fix typo in mdm.json

* Update src/services/mdm/MdmService.ts

Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>

---------

Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
cte pushed a commit that referenced this pull request Jun 24, 2025
* Fix typo in mdm.json

* Update src/services/mdm/MdmService.ts

Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>

---------

Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. lgtm This PR has been approved by a maintainer size:S This PR changes 10-29 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants