Skip to content

Conversation

@hussein-saad
Copy link

@hussein-saad hussein-saad commented Mar 17, 2025

Closes #47

Changes

Refactoring and Reorganization:

  • Split the resolveTypeNames function into multiple specialized functions for different types of resolution:
    • resolveDecoratorTypes
    • resolveModelTypes
    • resolveClassDeclarationTypes
    • resolveMapDeclarationTypes
    • resolveDecoratorArgumentTypes
    • resolvePropertyTypes
    • resolveScalarTypes

Documentation and Comments:

  • Added JSDoc comments to each new function to describe its purpose, parameters, and usage [1] [2].

Code Simplification:

  • Simplified the main resolveTypeNames function by delegating specific tasks to the newly created specialized functions.

Related Issues

Author Checklist

  • Ensure you provide a DCO sign-off for your commits using the --signoff option of git commit.
  • Vital features and changes captured in unit and/or integration tests
  • Commits messages follow AP format
  • Extend the documentation, if necessary
  • Merging to main from fork:branchname

…or decorators, models, classes, maps, and properties

Signed-off-by: Hussein Saad <[email protected]>
@hussein-saad
Copy link
Author

hi @sanketshevkar, could you please review this PR?

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.

Refactor the Type Resolution Logic in metamodelutil.js

1 participant