Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

pnpm run fetch errors with Failed to parse gherkin from ... #61

Closed
WesleyBatista opened this issue May 19, 2024 · 1 comment · Fixed by #62
Closed

pnpm run fetch errors with Failed to parse gherkin from ... #61

WesleyBatista opened this issue May 19, 2024 · 1 comment · Fixed by #62

Comments

@WesleyBatista
Copy link
Contributor

Getting the following while getting started with the repo:

...
 grammar  Fetched dax
 grammar  Fetched desktop
 grammar  Fetched elm
 grammar  Fetched erb
 grammar  Fetched elixir
 grammar  Fetched erlang
 grammar  Fetched fennel
Failed to parse gherkin from https://github.com/alexkrechik/VSCucumberAutoComplete/blob/fbb554addf81e8961b58b1577924af10fecf9694/gclient/syntaxes/feature.tmLanguage

file:///path/to/textmate-grammars-themes/node_modules/.pnpm/[email protected]/node_modules/js-yaml/dist/js-yaml.mjs:1273
  return new exception(message, mark);
         ^
YAMLException: end of the stream or a document separator is expected (210:75)

 207 |  ... 
 208 |  ... 
 209 |  ... 
 210 |  ... e-responsive" style="word-wrap: break-word;">
------------------------------------------^
 211 |  ... nv-production page-responsive" style="word-wrap: break-word;">
 212 |  ... 
    at generateError (file:///path/to/textmate-grammars-themes/node_modules/.pnpm/[email protected]/node_modules/js-yaml/dist/js-yaml.mjs:1273:10)
    at throwError (file:///path/to/textmate-grammars-themes/node_modules/.pnpm/[email protected]/node_modules/js-yaml/dist/js-yaml.mjs:1277:9)
    at readDocument (file:///path/to/textmate-grammars-themes/node_modules/.pnpm/[email protected]/node_modules/js-yaml/dist/js-yaml.mjs:2735:5)
    at loadDocuments (file:///path/to/textmate-grammars-themes/node_modules/.pnpm/[email protected]/node_modules/js-yaml/dist/js-yaml.mjs:2778:5)
    at Object.load$1 [as load] (file:///path/to/textmate-grammars-themes/node_modules/.pnpm/[email protected]/node_modules/js-yaml/dist/js-yaml.mjs:2804:19)
    at parseYaml (/path/to/textmate-grammars-themes/scripts/shared/parse.ts:20:15)
    at parseFile (/path/to/textmate-grammars-themes/scripts/shared/parse.ts:44:14)
    at fetchGrammar (/path/to/textmate-grammars-themes/scripts/grammars/fetch.ts:112:18)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at <anonymous> (/path/to/textmate-grammars-themes/scripts/grammars/fetch.ts:40:39) {
  reason: 'end of the stream or a document separator is expected',
  mark: {
    name: null,
    buffer: '\n' +
      '\n' +
      '\n' +
      '\n' +
      '\n' +
      '\n' +
      '<!DOCTYPE html>\n' +
      '<html\n' +
@WesleyBatista
Copy link
Contributor Author

This url pointing to master is returning 404:

textmate-grammars-themes/sources-grammars.ts

Line 575 in 0919d20

source: 'https://github.com/alexkrechik/VSCucumberAutoComplete/blob/master/gclient/syntaxes/feature.tmLanguage',

There were recent changes in that repo, moving the file from gclient/syntaxes/feature.tmLanguage to syntaxes/feature.tmLanguage.json:

Screenshot 2024-05-19 at 14 02 45

WesleyBatista added a commit to WesleyBatista/textmate-grammars-themes that referenced this issue May 19, 2024
@WesleyBatista
Update source for gherkin
dde64d9 
Closes shikijs#61
@WesleyBatista WesleyBatista mentioned this issue May 19, 2024
Merged
@antfu antfu closed this as completed in #62 May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: update source for gherkin WesleyBatista/textmate-grammars-themes
1 participant
@WesleyBatista