Skip to content

Fix for resolve paths in cli #16

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

toddpi314-reuters
Copy link

Ref: #14

The CLI project now properly handles imports in CML files by:

  1. Using CMLImportResolver to explicitly resolve imports
  2. Using absolute file paths to ensure correct URI resolution
  3. Updating test files to match DSL project's import syntax

This aligns the CLI's import handling with the DSL project's implementation.

…ile paths

The CLI project now properly handles imports in CML files by:
1. Using CMLImportResolver to explicitly resolve imports
2. Using absolute file paths to ensure correct URI resolution
3. Updating test files to match DSL project's import syntax

This aligns the CLI's import handling with the DSL project's implementation.
@stefan-ka
Copy link
Member

Hi @toddpi314-reuters

I had to fix some stuff in the pipeline; we haven't pushed here for a while :)

Could you please update your PR so it is rebased to the current state of our master branch?

Thanks and best regards,
Stefan

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants