Skip to content

Conversation

shortstacked
Copy link
Contributor

@shortstacked shortstacked commented Oct 21, 2025

Summary

Adds quotes around file paths in commands to ensure compatibility with paths containing spaces, especially on Windows systems. This prevents issues where the command interpreter misinterprets the space as a delimiter, leading to incorrect file path resolution.

Tested on a test workflow to ensure it passed and could build

image

Related Linear tickets, Github issues, and Community forum posts

closes #1514
#20251

Review / Merge checklist

  • PR title and summary are descriptive. (conventions)
  • Docs updated or follow-up ticket created.
  • Tests included.
  • PR Labeled with release/backport (if the PR is an urgent fix that needs to be backported)

@shortstacked shortstacked requested review from a team and alexgrozav and removed request for a team October 21, 2025 14:13
@n8n-assistant n8n-assistant bot added core Enhancement outside /nodes-base and /editor-ui n8n team Authored by the n8n team labels Oct 21, 2025
Copy link

codecov bot commented Oct 21, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

Copy link

currents-bot bot commented Oct 21, 2025

E2E Tests: n8n tests failed after 6m 47.2s

🟢 575 · 🔴 1 · ⚪️ 12 · 🟣 3

View Run Details

Run Details

  • Project: n8n

  • Groups: 3

  • Framework: Playwright

  • Run Status: Failed

  • Commit: 506faa6

  • Spec files: 95

  • Overall tests: 588

  • Duration: 6m 47.2s

  • Parallelization: 9

Failed Spec Files

Spec File Failures
tests/ui/7-workflow-actions.spec.ts 1

Groups

GroupId Results Spec Files Progress
ui 🟢 529 · 🔴 1 · ⚪️ 12 · 🟣 3 88 / 88
ui:isolated 🟢 32 · 🔴 0 · ⚪️ 0 5 / 5
No name 🟢 14 · 🔴 0 · ⚪️ 0 2 / 2


This message was posted automatically by currents.dev | Integration Settings

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Enhancement outside /nodes-base and /editor-ui n8n team Authored by the n8n team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

How to use plug-ins?

2 participants