Skip to content
This repository has been archived by the owner on May 8, 2024. It is now read-only.

Adding post 1990's Iorter and moving the db unit test #477

Merged
merged 9 commits into from
Feb 23, 2024
Merged

Conversation

BobBorges
Copy link
Collaborator

Here come the last iorter and an update unit test per #294

Also moved the input files for the db unittest in accordance with plans made re #329

@MansMeg MansMeg requested a review from ninpnin February 20, 2024 17:40
Copy link
Collaborator

@ninpnin ninpnin left a comment

Choose a reason for hiding this comment

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

I think we should remove hard-coded paths and use argparse arguments if possible

@BobBorges
Copy link
Collaborator Author

just in the scripts/stats-dashboard/mp-coverage.py, or you have something else in mind? I actually wanted to move almost everything from that stats dashboard process to the pyriksdagen module.

@ninpnin
Copy link
Collaborator

ninpnin commented Feb 23, 2024

I thought just here, since we make changes to it now.

BTW there is a merge conflict in test/pytestconfig.py now, can you fix that?

@BobBorges
Copy link
Collaborator Author

OK, yep, I have it open now.

@BobBorges BobBorges merged commit 5494fd6 into dev Feb 23, 2024
12 checks passed
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.

2 participants