Skip to content

Use @iarna/toml instead of toml#21

Open
jloehel wants to merge 1 commit intoyaegassy:masterfrom
jloehel:wip-fix-toml
Open

Use @iarna/toml instead of toml#21
jloehel wants to merge 1 commit intoyaegassy:masterfrom
jloehel:wip-fix-toml

Conversation

@jloehel
Copy link
Copy Markdown

@jloehel jloehel commented Apr 15, 2025

The pyproject.toml is fine. But npms toml complains. Error Message during parsing pyproject.toml:

./node_modules/toml/lib/parser.js:3833
      throw peg$buildException(null, peg$maxFailExpected, peg$maxFailPos);
      ^
[SyntaxError: Expected "}", [ \t] or [A-Za-z0-9_\-] but "\"" found.] {
  expected: [
    { type: 'literal', value: '}', description: '"}"' },
    { type: 'class', value: '[ \\t]', description: '[ \\t]' },
    {
      type: 'class',
      value: '[A-Za-z0-9_\\-]',
      description: '[A-Za-z0-9_\\-]'
    }
  ],
  found: '"',
  offset: 113,
  line: 6,
  column: 6
}

See: BinaryMuse/toml-node#51 Does not look like that it get fixed soon.

See: BinaryMuse/toml-node#51
Does not look like that it gets fixed soon.

Signed-off-by: Jürgen Löhel <2422012+jloehel@users.noreply.github.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