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

v0.11.0 #433

Merged
merged 117 commits into from
Dec 14, 2023
Merged

v0.11.0 #433

merged 117 commits into from
Dec 14, 2023

Conversation

ninpnin
Copy link
Collaborator

@ninpnin ninpnin commented Dec 13, 2023

No description provided.

github-actions bot and others added 30 commits September 11, 2023 10:53
All "paragraphs" have an ID : unit test.
…ot-sort

sort maj.min.patch with ints higher than 9
test: extend testing to each year in the data
…n-mp

add baseline number of MPs per parliament year/chamber/session
depreciates scripts/git-add_QC-sample.sh
organized diff-sampling stuff into subdir
(re)curate empty protocols in 202122 year
@ninpnin ninpnin requested review from MansMeg and BobBorges December 13, 2023 19:31
@MansMeg
Copy link
Collaborator

MansMeg commented Dec 13, 2023

Should we release it or try to merge the last prs to dev first. Rm r package make sense since its small.

@ninpnin
Copy link
Collaborator Author

ninpnin commented Dec 13, 2023

I don't think it makes sense to delay a release until we sort out a test technicality, if making releases is cheap

@MansMeg
Copy link
Collaborator

MansMeg commented Dec 13, 2023

I agree. We want rapid releases.

Copy link
Collaborator

@MansMeg MansMeg left a comment

Choose a reason for hiding this comment

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

Its hard to review this full release since there are too many files. But since the reviews going into dev has been reviewed Im happy to approve.

test is obsolete and doesn't actually check anything.
@BobBorges
Copy link
Collaborator

Test pass and everything has been reviewed.
image
maybe merge #356 and #431 into dev first (if the test pass on the last pushes).

@MansMeg
Copy link
Collaborator

MansMeg commented Dec 14, 2023

Alright! Maybe just release it and do another quick release with the other PRs?

@BobBorges
Copy link
Collaborator

Dunno what happened with #356, but it's fixed.

@ninpnin ninpnin merged commit a5cc855 into main Dec 14, 2023
3 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.

4 participants