Skip to content

Implement Disposable Workspace in all Importers#100

Merged
tkukuljan merged 8 commits into
developfrom
Teo-develop
Nov 25, 2022
Merged

Implement Disposable Workspace in all Importers#100
tkukuljan merged 8 commits into
developfrom
Teo-develop

Conversation

@tkukuljan
Copy link
Copy Markdown
Contributor

#64

@tkukuljan tkukuljan added the enhancement New feature or request label Nov 23, 2022
@tkukuljan tkukuljan self-assigned this Nov 23, 2022
@tkukuljan tkukuljan marked this pull request as ready for review November 24, 2022 14:46
@tkukuljan
Copy link
Copy Markdown
Contributor Author

The pull request is finished by Andrea and me.

FYI: @dcolleoni @dtrzesniak @amuolo

@amuolo amuolo changed the title Teo develop Implement Disposable Workspace in all Importers Nov 24, 2022
@amuolo amuolo linked an issue Nov 24, 2022 that may be closed by this pull request
@amuolo amuolo requested a review from dtrzesniak November 24, 2022 20:54
Copy link
Copy Markdown
Contributor

@dcolleoni dcolleoni left a comment

Choose a reason for hiding this comment

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

You are reverting many changes that were done in previous PR.
You have most probably resolved conflicts in the wrong way.
Thanks for fixing this.

Comment thread ifrs17/Import/Importers.ipynb Outdated
Comment thread ifrs17/Import/Importers.ipynb Outdated
Comment thread ifrs17/Import/Importers.ipynb Outdated
@tkukuljan tkukuljan merged commit eea6596 into develop Nov 25, 2022
@tkukuljan tkukuljan deleted the Teo-develop branch November 25, 2022 09:38
dtrzesniak pushed a commit that referenced this pull request Nov 28, 2022
* Adding new workspaces where needed.

* Changing last Workspaces to workspaces.

* restore metadata

* fix data node state validation + cleanups

* fix text

* fix text part 2

Co-authored-by: Andrea Muolo <amuolo@systemorph.com>
dtrzesniak pushed a commit that referenced this pull request Nov 29, 2022
* Adding new workspaces where needed.

* Changing last Workspaces to workspaces.

* restore metadata

* fix data node state validation + cleanups

* fix text

* fix text part 2

Co-authored-by: Andrea Muolo <amuolo@systemorph.com>
@dcolleoni dcolleoni added this to the v1.1.0 milestone Apr 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

Introduce use of disposable Workspace in import methods

2 participants