Skip to content

Ownership - accept changes into master#3

Merged
dgunning merged 5 commits into
mainfrom
ownership
Jan 4, 2023
Merged

Ownership - accept changes into master#3
dgunning merged 5 commits into
mainfrom
ownership

Conversation

@dgunning
Copy link
Copy Markdown
Owner

@dgunning dgunning commented Jan 4, 2023

The changes in the ownership branch are finished and tested locally. Merge into master and continue testing

@dgunning dgunning merged commit 1f797df into main Jan 4, 2023
@dgunning dgunning deleted the ownership branch January 27, 2023 14:07
baqamisaif pushed a commit to baqamisaif/edgartools that referenced this pull request Dec 27, 2025
This commit addresses 6 priority improvements to edgar/llm.py:

- Fix Issue dgunning#1: extract_sections() now returns list when track_filtered=False
  (was incorrectly returning tuple, breaking API contract)
- Fix Issue dgunning#2: Refactor tools.llm_extraction to edgar.llm_extraction
  (moves extract_item_with_boundaries to packaged module)
- Fix Issue dgunning#3: Remove dead code _STATEMENT_KEYWORDS constant
- Fix Issue dgunning#4: Add comprehensive type hints to all private functions
- Fix Issue dgunning#5: Upgrade error messages from debug to warning level
- Fix Issue dgunning#6: Add unit tests for return type contracts

Breaking change fix: Users relying on default behavior now get list instead
of (list, {}) tuple, matching documented API contract.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
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.

1 participant