Skip to content

[SHIP-793] Allow tags to be returned in File Importers#504

Merged
dkolas merged 3 commits into
mainfrom
dave/ship-793-python-changes-for-add-tags-to-file-importer-output
Aug 14, 2023
Merged

[SHIP-793] Allow tags to be returned in File Importers#504
dkolas merged 3 commits into
mainfrom
dave/ship-793-python-changes-for-add-tags-to-file-importer-output

Conversation

@dkolas
Copy link
Copy Markdown
Contributor

@dkolas dkolas commented Jul 27, 2023

Depends on https://github.com/nludb/nludb/pull/600

Allow FileImporters to return tags in the RawDataPluginOutput.

Copy link
Copy Markdown
Contributor

@EniasCailliau EniasCailliau left a comment

Choose a reason for hiding this comment

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

LGTM 🔥

Copy link
Copy Markdown
Contributor

@douglas-reid douglas-reid left a comment

Choose a reason for hiding this comment

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

Looks good to me.

Not clear why the tests are failing tho.

@dkolas
Copy link
Copy Markdown
Contributor Author

dkolas commented Jul 27, 2023

Looks good to me.

Not clear why the tests are failing tho.

Tests will fail until the Engine pr is deployed to staging / prod.

@dkolas dkolas added this pull request to the merge queue Aug 14, 2023
Merged via the queue into main with commit 15bd58d Aug 14, 2023
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.

3 participants