Skip to content

feat: Add uhura#55

Merged
joloppo merged 31 commits into
v5from
v5-uhura
Jul 19, 2023
Merged

feat: Add uhura#55
joloppo merged 31 commits into
v5from
v5-uhura

Conversation

@joloppo
Copy link
Copy Markdown
Contributor

@joloppo joloppo commented Jul 17, 2023

Made all resources (except Keyed) into uhura readers/writers. Added many tests.

@joloppo joloppo marked this pull request as ready for review July 17, 2023 13:09
Comment thread demo/demo.py
Comment thread dynamicio/io/postgres.py Outdated
Copy link
Copy Markdown

@matteo-pallini matteo-pallini left a comment

Choose a reason for hiding this comment

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

Left a couple of comments (I didn't review everything yet though), but as discussed it may be worth do a bit of refactoring as well on Friday in person

Comment thread dynamicio/io/file/csv.py
Comment thread dynamicio/io/file/csv.py
Comment thread dynamicio/io/file/csv.py
@joloppo joloppo added echoes/effort: L High effort changes (M * 2) tech debt tech debt labels Jul 19, 2023
@joloppo joloppo merged commit d6bf630 into v5 Jul 19, 2023
@arturk-vortexa arturk-vortexa mentioned this pull request Aug 4, 2023
@joloppo joloppo deleted the v5-uhura branch September 20, 2023 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

echoes/effort: L High effort changes (M * 2) tech debt tech debt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants