Skip to content

Conversation

Synrom
Copy link
Contributor

@Synrom Synrom commented Aug 28, 2025

Type of Changes

Type
βœ“ πŸ› Bug fix

Description

Closes #2643

Copy link
Member

@Pierre-Sassoulas Pierre-Sassoulas left a comment

Choose a reason for hiding this comment

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

Look great thank you. But we should probably do something more before reaching the memory error to consider the issue fixed.

Copy link

codecov bot commented Aug 28, 2025

Codecov Report

βœ… All modified and coverable lines are covered by tests.
βœ… Project coverage is 93.31%. Comparing base (f7c8b4c) to head (cf2227a).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #2809   +/-   ##
=======================================
  Coverage   93.31%   93.31%           
=======================================
  Files          91       91           
  Lines       11094    11094           
=======================================
  Hits        10352    10352           
  Misses        742      742           
Flag Coverage Ξ”
linux 93.17% <100.00%> (ΓΈ)
pypy 93.31% <100.00%> (ΓΈ)
windows 93.29% <100.00%> (ΓΈ)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Ξ”
astroid/builder.py 95.13% <100.00%> (ΓΈ)
πŸš€ New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Pierre-Sassoulas
Copy link
Member

Co-authored-by: Patcher [email protected] ?

@Pierre-Sassoulas
Copy link
Member

Should I keep this in the commit @Synrom ? (Look like a misconfigured git locally)

@Synrom Synrom force-pushed the 2643-MemoryError-when-parsing-deeply-nested-parentheses branch from 8ad7cdf to cf2227a Compare August 28, 2025 13:00
@Synrom
Copy link
Contributor Author

Synrom commented Aug 28, 2025

Sorry, I missed that. Fixed it now. Thanks for catching it! πŸ™

@Pierre-Sassoulas Pierre-Sassoulas merged commit 1327f6b into pylint-dev:main Aug 28, 2025
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MemoryError when parsing deeply nested parentheses
2 participants