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.

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

Boundaries issue #9874

Closed
1 task
NormandoHall opened this issue Feb 3, 2025 · 0 comments · Fixed by #9886
Closed
1 task

Boundaries issue #9874

NormandoHall opened this issue Feb 3, 2025 · 0 comments · Fixed by #9886
Assignees
Labels
kind: bug Something isn't working

Comments

@NormandoHall
Copy link

NormandoHall commented Feb 3, 2025

Verify canary release

  • I verified that the issue exists in the latest Turborepo canary release.

Link to code that reproduces this issue

npx create-turbo@latest -e with-shell-commands

Which canary version will you have in your reproduction?

2.4.0

Enviroment information

CLI:
   Version: 2.4.0
   Path to executable: /opt/webnews/devel/node_modules/turbo-linux-64/bin/turbo
   Daemon status: Running
   Package manager: npm

Platform:
   Architecture: x86_64
   Operating system: linux
   WSL: false
   Available memory (MB): 1739
   Available CPU cores: 4

Environment:
   CI: None
   Terminal (TERM): xterm-256color
   Terminal program (TERM_PROGRAM): vscode
   Terminal program version (TERM_PROGRAM_VERSION): 1.96.4
   Shell (SHELL): /bin/bash
   stdin: false

Expected behavior

Runs normal

Actual behavior

Get a los of warnings like these:

 WARNING  Lexer error at 44028..44028
 WARNING  Lexer error at 44118..44118
 WARNING  Lexer error at 44183..44183

and random errors:

error: Expected '}', got ';'
  --> /opt/webnews/devel/apps/editor/src/lib/components/Map/components/ui/ToolTip.svelte:33:49
   |
33 |     leafletToolTip = layerComponent.mountLayer();
   |     

To Reproduce

turbo boundaries

Additional context

No response

@NormandoHall NormandoHall added kind: bug Something isn't working needs: triage New issues get this label. Remove it after triage labels Feb 3, 2025
@tknickman tknickman removed the needs: triage New issues get this label. Remove it after triage label Feb 3, 2025
NicholasLYang added a commit that referenced this issue Feb 4, 2025
### Description

SWC currently cannot parse svelte or vue. Therefore we ignore them when
it comes to boundaries, but we do emit a warning for users. Note that we
specifically do *not* emit a warning per file, since that could be a lot
of warnings. We do however emit per package, but that should hopefully
not be too excessive

closes #9874 

### Testing Instructions

<!--
  Give a quick description of steps to test your changes.
-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind: bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants