Skip to content

Commit

Permalink
Use alpine 3.21 (CMake 3.21), bump tests (#56)
Browse files Browse the repository at this point in the history
Modify code to work with the latest tests. The only change is to remove length limits of section, property and value.
  • Loading branch information
xuhdev authored Dec 12, 2024
1 parent ccfc270 commit ef927af
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 11 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/runtime.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ jobs:
test:
name: Runtime
runs-on: ubuntu-latest
container: python:3.13-alpine3.20
container: python:3.13-alpine3.21

steps:
- name: Install Alpine Dependencies
Expand Down
9 changes: 0 additions & 9 deletions editorconfig/ini.py
Original file line number Diff line number Diff line change
Expand Up @@ -27,10 +27,6 @@

__all__ = ["ParsingError", "EditorConfigParser"]

MAX_SECTION_LENGTH = 4096
MAX_PROPERTY_LENGTH= 50
MAX_VALUE_LENGTH = 255


class EditorConfigParser(object):

Expand Down Expand Up @@ -138,8 +134,6 @@ def _read(self, fp, fpname):
mo = self.SECTCRE.match(line)
if mo:
sectname = mo.group('header')
if len(sectname) > MAX_SECTION_LENGTH:
continue
in_section = True
matching_section = self.matches_filename(fpname, sectname)
# So sections can't start with a continuation line
Expand All @@ -160,9 +154,6 @@ def _read(self, fp, fpname):
if optval == '""':
optval = ''
optname = self.optionxform(optname.rstrip())
if (len(optname) > MAX_PROPERTY_LENGTH or
len(optval) > MAX_VALUE_LENGTH):
continue
if not in_section and optname == 'root':
self.root_file = (optval.lower() == 'true')
if matching_section:
Expand Down

0 comments on commit ef927af

Please sign in to comment.