georgetown-cset / parat

🦜 PARAT: CSET's Private-sector AI-Related Activity Tracker

https://parat.cset.tech

Other

4 stars 0 forks source link

Fix incorrect slugification of company names #397

Closed brianlove closed 3 weeks ago

brianlove commented 1 month ago

Ignore slashes and periods when slugifying company names.

Closes #365

github-actions[bot] commented 1 month ago

No need for rebasing :+1: behind_count is 0 ahead_count is 2

github-actions[bot] commented 1 month ago

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status

594 423 71% 0% 🟢

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: b0305e9 by action🐍

Lines	Covered	Coverage	Threshold	Status
594	423	71%	0%	🟢

github-actions[bot] commented 1 month ago

JavaScript Coverage

Summary

Lines	Statements	Branches	Functions
	67.67% (404/597)	55.18% (213/386)	68.27% (127/186)

Modified Files • (67%)

File	% Stmts	% Branch	% Funcs	% Lines	Uncovered Line #s
All files	67.67	55.18	68.27	67.43
util	83.51	52.72	84.37	86.58
index.js	83.33	55.55	70	92.85	47

brianlove commented 4 weeks ago

@jmelot Found and fixed the issue - Gatsby used a different slugification process, so there wasn't agreement between the pages that were actually generated and the links that were generated.