Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ability to exclude pairs of ids for matching #38

Merged
merged 12 commits into from
Mar 1, 2024

Conversation

jmelot
Copy link
Member

@jmelot jmelot commented Jan 31, 2024

Closes https://github.com/georgetown-cset/cset_article_schema/issues/135

James I am going to see if we have time to do a little live code review of this (untested, other than the little unit test) method during our 1-1. It should prevent two articles from being directly linked, but they might still be transitively linked.

Copy link

github-actions bot commented Jan 31, 2024

No need for rebasing 👍
behind_count is 0
ahead_count is 12

Copy link

github-actions bot commented Jan 31, 2024

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
300 260 87% 0% 🟢

New Files

File Coverage Status
tests/test_make_unlink_rows.py 100% 🟢
utils/make_unlink_rows.py 48% 🟢
TOTAL 74% 🟢

Modified Files

File Coverage Status
tests/test_create_merge_ids.py 98% 🟢
utils/create_merge_ids.py 92% 🟢
TOTAL 95% 🟢

updated for commit: dfd8fe3 by action🐍

@jmelot jmelot requested review from niharikasingh and removed request for jamesdunham February 13, 2024 17:28
@jmelot jmelot merged commit 4e053bf into 35-cloud-composer-2-and-sensors Mar 1, 2024
4 checks passed
@jmelot jmelot deleted the 135-remove-fp branch March 1, 2024 16:23
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.

2 participants