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

feat(eyecite): Fix california style #238

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

flooie
Copy link
Contributor

@flooie flooie commented Mar 6, 2025

These complex styles were failing with square brackets and years before the citation. Also addressed
an issue with extra running too long and
simplified the check for parallel citations

Meant to address complex citation patterns #185

This does not address California parentheticals

These complex styles were failing with square brackets
and years before the citation.  Also addressed
an issue with extra running too long and
simplified the check for parallel citations
@flooie flooie requested a review from grossir March 6, 2025 16:42
Change type hint
move is parallel to fullcase citation
Copy link
Contributor

github-actions bot commented Mar 6, 2025

The Eyecite Report 👁️

Gains and Losses

There were 0 gains and 6 losses.

Click here to see details.
id Gain Loss
2257892 Misura
2020059 Holloway
2020059 Rogers
2020059 Davis
2123399 Rose
3692222 Huson

Time Chart

image

Generated Files

Branch 1 Output
Branch 2 Output
Full Output CSV

Copy link
Contributor

@grossir grossir left a comment

Choose a reason for hiding this comment

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

This is working fine; just a few clean up comments

@@ -314,3 +314,6 @@ def short_cite_re(regex):
\ ?
{PARENTHETICAL_REGEX}
"""

# A simple regex to check if year precedes citation
DEFENDNAT_YEAR_REGEX = r"(?P<defendant>.*)\s\((?P<year>\d{4})\)$"
Copy link
Contributor

Choose a reason for hiding this comment

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

typo: DEFENDNAT_YEAR_REGEX
should be DEFENDANT_YEAR_REGEX

)

# d this conclusion is supported by People v. Beach (1983) 147 Cal.App.3d 612 [195 Cal.Rptr. 381] (relocation away from home community as
Copy link
Contributor

Choose a reason for hiding this comment

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

unused, should be removed?


Returns: None
"""
if self.full_span_start == preceding.full_span_start:
Copy link
Contributor

Choose a reason for hiding this comment

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

you are removing the self.full_span_end == preceding.full_span_end condition, is that desired?

@grossir grossir assigned flooie and unassigned grossir Mar 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: PRs to Review
Development

Successfully merging this pull request may close these issues.

2 participants