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(csharp,java): support aliasing imports #115

Merged

Conversation

salujajaskeerat
Copy link
Contributor

@salujajaskeerat salujajaskeerat commented Jul 31, 2024

Closes #114

This PR would support the codegen for CSharp
This PR depends uses the API which is recently added to the concerto-core via PR accordproject/concerto#890 . Once this PR is merged to concerto-core the current Pr would be functional.

Related PR

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 4 commits August 1, 2024 00:18
- Earlier non-imported types gave error.

Signed-off-by: Jaskeerat Singh Saluja <[email protected]>
@salujajaskeerat salujajaskeerat changed the title Codegen CSharp support for aliasing imports Codegen [CSharp + Java] support for aliasing imports Aug 4, 2024
@ekarademir
Copy link
Contributor

Can you update the title of the commit to follow the conventional commit style please?

https://www.conventionalcommits.org/en/v1.0.0/#summary

Copy link
Contributor

@ekarademir ekarademir left a comment

Choose a reason for hiding this comment

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

could you run test:updateSnapshots and add the result to the PR please?

@salujajaskeerat salujajaskeerat changed the title Codegen [CSharp + Java] support for aliasing imports feat(codegen/csharp,codegen/java): add support for aliasing imports Aug 6, 2024
Signed-off-by: Jaskeerat Singh Saluja <[email protected]>
@salujajaskeerat salujajaskeerat changed the title feat(codegen/csharp,codegen/java): add support for aliasing imports feat(csharp,java): support aliasing imports Aug 23, 2024
Jaskeerat Singh Saluja added 2 commits September 3, 2024 16:16
@salujajaskeerat
Copy link
Contributor Author

Below contains the code-coverage reports from the changes made in the concerto-codegen for csharp and java

Screenshot 2024-09-03 at 4 30 56 PM
Screenshot 2024-09-03 at 4 31 14 PM
Screenshot 2024-09-03 at 4 31 32 PM
Screenshot 2024-09-03 at 4 31 51 PM
Screenshot 2024-09-03 at 4 32 25 PM

Signed-off-by: Jaskeerat Singh Saluja <[email protected]>
lib/codegen/fromcto/csharp/csharpvisitor.js Outdated Show resolved Hide resolved
lib/codegen/fromcto/java/javavisitor.js Outdated Show resolved Hide resolved
Signed-off-by: Jaskeerat Singh Saluja <[email protected]>
@coveralls
Copy link

Coverage Status

coverage: 98.773% (+0.005%) from 98.768%
when pulling c204762 on salujajaskeerat:sahib/i114/csharp-alias-import
into 15afc18 on accordproject:main.

@mttrbrts mttrbrts dismissed ekarademir’s stale review September 4, 2024 08:53

Feedback addressed. Snapshots have been updated

@mttrbrts mttrbrts merged commit e4968a6 into accordproject:main Sep 4, 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.

concerto-codegen supports import aliasing for [CSharp + Java]
5 participants