Skip to content

acc: Clean up unnecessary replacements - #3195

Merged
denik merged 8 commits into
mainfrom
denik/cleanup-slashes-repls
Jul 9, 2025
Merged

acc: Clean up unnecessary replacements#3195
denik merged 8 commits into
mainfrom
denik/cleanup-slashes-repls

Conversation

@denik

@denikdenik commented Jul 8, 2025

Copy link
Copy Markdown
Contributor

Since #3175 the diagnostics are rendered with forward slashes on Windows.

@denik
deniktemporarily deployed to test-trigger-is July 8, 2025 14:17 — with GitHub Actions Inactive
@denik
deniktemporarily deployed to test-trigger-is July 8, 2025 14:21 — with GitHub Actions Inactive
@eng-dev-ecosystem-bot

eng-dev-ecosystem-bot commented Jul 8, 2025

Copy link
Copy Markdown
Collaborator

Run: 16163393817

Env✅‌pass🙈‌skip
✅‌aws linux306379
✅‌aws windows307378
✅‌aws-ucws linux412279
✅‌aws-ucws windows413278
✅‌azure linux306378
✅‌azure windows307377
✅‌azure-ucws linux414276
✅‌azure-ucws windows415275
✅‌gcp linux305380
✅‌gcp windows306379

@denik
denikforce-pushed the denik/cleanup-slashes-repls branch from a3fc7fe to c5ade2fCompareJuly 8, 2025 14:34
@denik
deniktemporarily deployed to test-trigger-is July 8, 2025 14:34 — with GitHub Actions Inactive
@denik
denik marked this pull request as ready for review July 8, 2025 14:50
@denikdenik changed the title WIP - clean up slashes replsacc: Clean up unnecessary replacementsJul 8, 2025
@denik
denikforce-pushed the denik/cleanup-slashes-repls branch from c5ade2f to fc7b63cCompareJuly 8, 2025 15:44
@denik
deniktemporarily deployed to test-trigger-is July 8, 2025 15:44 — with GitHub Actions Inactive
@denik
deniktemporarily deployed to test-trigger-is July 9, 2025 07:42 — with GitHub Actions Inactive
@denik
denik added this pull request to the merge queueJul 9, 2025
Merged via the queue into main with commit 40fae91Jul 9, 2025
13 checks passed
@denik
denik deleted the denik/cleanup-slashes-repls branch July 9, 2025 08:34
shreyas-goenka pushed a commit that referenced this pull request Jul 9, 2025
shreyas-goenka added a commit that referenced this pull request Jul 10, 2025
## Changes
Reverts: #3195
## Why
These tests started failing because of the missing repl. Diff:
```
->>> cp [TESTROOT]/bundle/resources/clusters/run/spark_python_task/../../deploy/simple/hello_world.py .
+>>> cp [TESTROOT]\bundle\resources\clusters\run\spark_python_task/../../deploy/simple/hello_world.py .
```
denik added a commit that referenced this pull request May 20, 2026
Since #3175 the diagnostics are rendered with forward slashes on
Windows.
denik added a commit that referenced this pull request May 20, 2026
denik pushed a commit that referenced this pull request May 20, 2026
## Changes
Reverts: #3195
## Why
These tests started failing because of the missing repl. Diff:
```
->>> cp [TESTROOT]/bundle/resources/clusters/run/spark_python_task/../../deploy/simple/hello_world.py .
+>>> cp [TESTROOT]\bundle\resources\clusters\run\spark_python_task/../../deploy/simple/hello_world.py .
```
Sign up for freeto 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.

3 participants

@denik@eng-dev-ecosystem-bot@shreyas-goenka