Skip to content

Handle AttributeError subclasses with from_attributes#13096

Merged
Viicos merged 1 commit intomainfrom
vp/from-attributes-attributeerror-sub
Apr 20, 2026
Merged

Handle AttributeError subclasses with from_attributes#13096
Viicos merged 1 commit intomainfrom
vp/from-attributes-attributeerror-sub

Conversation

@Viicos
Copy link
Copy Markdown
Member

@Viicos Viicos commented Apr 20, 2026

Change Summary

Fixes #13092

Related issue number

Checklist

  • The pull request title is a good summary of the changes - it will be used in the changelog
  • Unit tests for the changes exist
  • Tests pass on CI
  • Documentation reflects the changes where applicable
  • My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

@Viicos Viicos added relnotes-fix Used for bugfixes. backport-2.13 Needs backport to 2.13 labels Apr 20, 2026
@cloudflare-workers-and-pages
Copy link
Copy Markdown

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Apr 20, 2026

Merging this PR will improve performance by 6.66%

⚠️ Different runtime environments detected

Some benchmarks with significant performance changes were compared across different runtime environments,
which may affect the accuracy of the results.

Open the report in CodSpeed to investigate

⚡ 1 improved benchmark
✅ 214 untouched benchmarks

Performance Changes

Benchmark BASE HEAD Efficiency
test_field_function_validator 343.3 µs 321.8 µs +6.66%

Comparing vp/from-attributes-attributeerror-sub (d80a022) with main (a6bf50b)

Open in CodSpeed

@github-actions
Copy link
Copy Markdown
Contributor

Coverage report

This PR does not seem to contain any modification to coverable code.

alokshukla631

This comment was marked as spam.

@Viicos Viicos merged commit 5bc4d9c into main Apr 20, 2026
79 checks passed
@Viicos Viicos deleted the vp/from-attributes-attributeerror-sub branch April 20, 2026 13:40
Viicos added a commit that referenced this pull request Apr 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-2.13 Needs backport to 2.13 relnotes-fix Used for bugfixes.

Projects

None yet

3 participants