Skip to content
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

feat(alias-import): added test case for introspector #882

Merged

Conversation

salujajaskeerat
Copy link
Contributor

@salujajaskeerat salujajaskeerat commented Jul 18, 2024

Closes #867 #878

Changes

  • Alias import test-cases for introspector test module

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

Jaskeerat Singh Saluja added 3 commits July 18, 2024 09:51
Signed-off-by: Jaskeerat Singh Saluja <[email protected]>
Signed-off-by: Jaskeerat Singh Saluja <[email protected]>
Copy link
Contributor

@dselman dselman left a comment

Choose a reason for hiding this comment

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

Looks good! Couple of small changes.

packages/concerto-core/test/introspect/introspector.js Outdated Show resolved Hide resolved
packages/concerto-core/test/introspect/introspector.js Outdated Show resolved Hide resolved
Signed-off-by: Jaskeerat Singh Saluja <[email protected]>
@mttrbrts mttrbrts requested a review from dselman July 24, 2024 09:12
Signed-off-by: Jaskeerat Singh Saluja <[email protected]>
@salujajaskeerat
Copy link
Contributor Author

Screenshot 2024-08-23 at 4 51 32 PM

@ekarademir
Copy link
Contributor

@dselman your comments have been addressed and since this is a dependency on other work for aliasing, and you are on leave, I'll dismiss the review on paper and merge the MR. Apologies for that.

@ekarademir ekarademir dismissed dselman’s stale review August 27, 2024 13:46

The MR is blocking other work and Dan is on leave. Dan's remarks have been addressed.

@ekarademir ekarademir merged commit fea4c91 into accordproject:main Aug 27, 2024
11 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.

Update concerto-core/serializer to understand aliased imports
4 participants