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

Feature mma 465 #477

Merged
merged 11 commits into from
Jul 15, 2021
Merged

Feature mma 465 #477

merged 11 commits into from
Jul 15, 2021

Conversation

holiday-sunrise
Copy link
Contributor

No description provided.

@@ -0,0 +1,110 @@
/*
Copy link
Contributor

@mlischetti mlischetti Jul 14, 2021

Choose a reason for hiding this comment

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

This changes are not need to fix the issue. Please remove Parent.java file from PR

@@ -156,6 +156,24 @@ public void setArtifactId(String artifactId) {
model.setArtifactId(artifactId);
}

/**
Copy link
Contributor

Choose a reason for hiding this comment

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

This changes are not need to fix the issue, please remove it

Copy link
Contributor

@mlischetti mlischetti left a comment

Choose a reason for hiding this comment

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

Hi @holiday-sunrise thanks for contributing to mma project.
Can you please isolate the core changes involved in this issue to make it mergeable?

@holiday-sunrise
Copy link
Contributor Author

Files have been removed

@mlischetti
Copy link
Contributor

mlischetti commented Jul 14, 2021

@holiday-sunrise, there is a misunderstood about PomModel.java file.
What I was trying to said is that changes in PomModel.java are not needed not that you should remove the file itself.

Rainer Drexler added 2 commits July 14, 2021 16:58
@mlischetti
Copy link
Contributor

PomModel.java file still contains unnecessary modification please revert this changes

@holiday-sunrise
Copy link
Contributor Author

Pom model should now reverted. sorry my mistake

Copy link
Contributor

@mlischetti mlischetti left a comment

Choose a reason for hiding this comment

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

LGTM

@mlischetti mlischetti merged commit d101d7a into mulesoft:master Jul 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

HttpConnectorRequester line 240 request builder is removed too early from parent
2 participants