Skip to content
This repository has been archived by the owner on Mar 21, 2021. It is now read-only.

Add returnMergedEntity option #439

Merged
merged 1 commit into from Apr 5, 2020
Merged

Add returnMergedEntity option #439

merged 1 commit into from Apr 5, 2020

Conversation

mshima
Copy link
Member

@mshima mshima commented Mar 13, 2020

Please make sure the below checklist is followed for Pull Requests.

  • Checks are green
  • Tests are added where necessary
  • Documentation is added/updated where necessary
  • Coding Rules & Commit Guidelines as per our CONTRIBUTING.md document are followed

Copy link
Member

@MathieuAA MathieuAA left a comment

Choose a reason for hiding this comment

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

I don't see any tests

lib/exporters/jhipster_deployment_exporter.js Outdated Show resolved Hide resolved
lib/exporters/jhipster_application_exporter.js Outdated Show resolved Hide resolved
lib/exporters/jhipster_entity_exporter.js Outdated Show resolved Hide resolved
lib/exporters/jhipster_entity_exporter.js Outdated Show resolved Hide resolved
lib/exporters/jhipster_application_exporter.js Outdated Show resolved Hide resolved
Copy link
Member

@MathieuAA MathieuAA left a comment

Choose a reason for hiding this comment

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

Tell me if you have any issue testing

lib/exporters/jhipster_application_exporter.js Outdated Show resolved Hide resolved
lib/exporters/jhipster_application_exporter.js Outdated Show resolved Hide resolved
lib/exporters/export_utils.js Outdated Show resolved Hide resolved
lib/jdl/jdl_importer.js Outdated Show resolved Hide resolved
@mshima mshima force-pushed the master branch 2 times, most recently from 2fcd069 to 3bf72ea Compare March 13, 2020 18:10
@MathieuAA
Copy link
Member

Can you finish this PR please @mshima?

@mshima
Copy link
Member Author

mshima commented Mar 26, 2020

@MathieuAA sure. I will try later today.

@mshima
Copy link
Member Author

mshima commented Mar 26, 2020

Probably I will split in smaller PRs.

@MathieuAA
Copy link
Member

Thanks :) Do as you please!

@mshima mshima force-pushed the master branch 2 times, most recently from 281eb88 to 122d30b Compare March 26, 2020 18:13
@mshima
Copy link
Member Author

mshima commented Mar 26, 2020

@MathieuAA first round is ready.

@MathieuAA
Copy link
Member

Can we close this one then?

@mshima
Copy link
Member Author

mshima commented Mar 26, 2020

Can we close this one then?

There are 2 more changes.

@mshima mshima force-pushed the master branch 2 times, most recently from eee8bf3 to 9bbcb32 Compare March 27, 2020 02:05
@mshima mshima changed the title Add option to don't write files to disk and others improvements. Add returnMergedEntity option Mar 27, 2020
@mshima
Copy link
Member Author

mshima commented Mar 27, 2020

@MathieuAA this is the last one for now.

.gitignore Outdated Show resolved Hide resolved
lib/exporters/jhipster_entity_exporter.js Outdated Show resolved Hide resolved
lib/exporters/jhipster_entity_exporter.js Outdated Show resolved Hide resolved
@mshima
Copy link
Member Author

mshima commented Mar 29, 2020

I just noticed that the only method that take subFolder (application.name) into account is writeEntities.
Am I wrong or should we change that?

@MathieuAA
Copy link
Member

I just noticed that the only method that take subFolder (application.name) into account is writeEntities.
Am I wrong or should we change that?

@mshima sorry didn't see your comment... Anyway, this is good to go, thanks :)

@MathieuAA MathieuAA merged commit e8fd743 into jhipster:master Apr 5, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants