Skip to content

Move files from main app to api project #1

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

Merged
merged 12 commits into from
Feb 8, 2022
Merged

Conversation

akshay-ap
Copy link
Collaborator

No description provided.

@akshay-ap akshay-ap changed the title Development Move files from main app to api project Feb 1, 2022
Copy link
Member

@ghareeb-falazi ghareeb-falazi left a comment

Choose a reason for hiding this comment

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

Please consider the comments

pom.xml Outdated
<artifactId>core</artifactId>
<version>4.5.6</version>
</dependency>
<dependency>
Copy link
Member

Choose a reason for hiding this comment

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

Could you make sure that all dependencies in this pom file are actually used? For example, intuitively I think that rpc.server is not needed by this repository, but I might be wrong

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, all the added dependencies are used in the project. Although, I have added exclusions as they were in the main app repository.

Copy link
Member

@ghareeb-falazi ghareeb-falazi left a comment

Choose a reason for hiding this comment

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

Everything looks good to me at this stage.

@ghareeb-falazi ghareeb-falazi merged commit e8b3504 into main Feb 8, 2022
@ghareeb-falazi ghareeb-falazi deleted the development branch February 8, 2022 11:47
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.

2 participants