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

fix(connector-corda): codegen missing prior removal of files #1559

Closed
petermetz opened this issue Nov 23, 2021 · 0 comments · Fixed by #1562
Closed

fix(connector-corda): codegen missing prior removal of files #1559

petermetz opened this issue Nov 23, 2021 · 0 comments · Fixed by #1562
Assignees
Labels
bug Something isn't working Corda dependencies Pull requests that update a dependency file P4 Priority 4: Low

Comments

@petermetz
Copy link
Member

Describe the bug

Based on this discussion 1 with @elenaizaguirre it seems like the files are not being deleted prior to them being regenerated which most likely lead to the strange situation we've found ourselves where files that are no longer being generated by the generator are still hanging around in/under version control simply because they were never deleted.

To Reproduce

Delete the codegen kotlin code, run a build and observe that there is a git diff which there should be none if this was working fine.

Expected behavior

Deleting generated code and then running the build should always produce equivalent, consistent code.

Hyperledger Cactus release version or commit (git rev-parse --short HEAD):

1.0.0-rc.2

Hyperledger Cactus Plugins/Connectors Used

  • Corda

Additional context

Came up during: #1294
more specifically
#1294 (comment)

@petermetz petermetz added bug Something isn't working Corda dependencies Pull requests that update a dependency file P4 Priority 4: Low labels Nov 23, 2021
@elenaizaguirre elenaizaguirre self-assigned this Nov 23, 2021
elenaizaguirre added a commit to elenaizaguirre/cactus that referenced this issue Nov 23, 2021
folders in connector-corda.

call this new script in generate-server script, before call
openapi-generator-cli

Closes: hyperledger#1559

Signed-off-by: Elena Izaguirre <e.izaguirre.equiza@accenture.com>
elenaizaguirre added a commit to elenaizaguirre/cactus that referenced this issue Nov 23, 2021
generate-server script calls this new script before call
openapi-generator-cli

Closes: hyperledger#1559

Signed-off-by: Elena Izaguirre <e.izaguirre.equiza@accenture.com>
elenaizaguirre added a commit to elenaizaguirre/cactus that referenced this issue Nov 26, 2021
added new pattern to 'clean' script to remove autogenerated files
from corda main-server

Closes: hyperledger#1559

Signed-off-by: Elena Izaguirre <e.izaguirre.equiza@accenture.com>
elenaizaguirre added a commit to elenaizaguirre/cactus that referenced this issue Nov 29, 2021
added new pattern to 'clean' script to remove autogenerated files
from corda main-server

Closes: hyperledger#1559

Signed-off-by: Elena Izaguirre <e.izaguirre.equiza@accenture.com>
elenaizaguirre added a commit to elenaizaguirre/cactus that referenced this issue Dec 1, 2021
added new pattern to 'clean' script to remove autogenerated files
from corda main-server

Closes: hyperledger#1559

Signed-off-by: Elena Izaguirre <e.izaguirre.equiza@accenture.com>
petermetz pushed a commit that referenced this issue Dec 1, 2021
added new pattern to 'clean' script to remove autogenerated files
from corda main-server

Closes: #1559

Signed-off-by: Elena Izaguirre <e.izaguirre.equiza@accenture.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Corda dependencies Pull requests that update a dependency file P4 Priority 4: Low
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants