Skip to content

fix(linter): Fix cloning on Windows by reducing the size of a problematic snapshot filename.#20511

Merged
graphite-app[bot] merged 1 commit intomainfrom
skill-issue
Mar 19, 2026
Merged

fix(linter): Fix cloning on Windows by reducing the size of a problematic snapshot filename.#20511
graphite-app[bot] merged 1 commit intomainfrom
skill-issue

Conversation

@connorshea
Copy link
Copy Markdown
Member

It seems fine to snapshot these separately so they're not creating such a lengthy snapshot name? As far as I can tell it's the same exact behavior as before. Fixes #20470.

@connorshea connorshea requested a review from camc314 as a code owner March 19, 2026 00:27
@github-actions github-actions bot added A-linter Area - Linter A-cli Area - CLI C-bug Category - Bug labels Mar 19, 2026
Copy link
Copy Markdown
Member

@leaysgur leaysgur left a comment

Choose a reason for hiding this comment

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

Would it be even better if there were comments? 👌🏻

@connorshea
Copy link
Copy Markdown
Member Author

Would it be even better if there were comments? 👌🏻

Added a comment to the test file, assuming that's what you meant? :)

@connorshea connorshea added the 0-merge Merge with Graphite Merge Queue label Mar 19, 2026
Copy link
Copy Markdown
Member Author

connorshea commented Mar 19, 2026

Merge activity

…atic snapshot filename. (#20511)

It seems fine to snapshot these separately so they're not creating such a lengthy snapshot name? As far as I can tell it's the same exact behavior as before. Fixes #20470.
@graphite-app graphite-app bot merged commit 04c84bb into main Mar 19, 2026
19 checks passed
@graphite-app graphite-app bot deleted the skill-issue branch March 19, 2026 01:00
@graphite-app graphite-app bot removed the 0-merge Merge with Graphite Merge Queue label Mar 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-cli Area - CLI A-linter Area - Linter C-bug Category - Bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Skill install on windows 11 fails with Filename too long

2 participants