Skip to content

Change install directory to ~/.wpilib or XDG_DATA_HOME/wpilib - #841

Merged
PeterJohnson merged 7 commits into
wpilibsuite:mainfrom
ThadHouse:installtodotfolder
Aug 28, 2026
Merged

PeterJohnson merged 7 commits into
wpilibsuite:mainfrom
ThadHouse:installtodotfolder

Conversation

@ThadHouse

Copy link
Copy Markdown
Member

On linux, falls back to ~/.wpilib if XDG_DATA_HOME doesn't exist.

Comment thread src/main/java/org/wpilib/gradlerio/wpi/WPIExtension.java Outdated
@sciencewhiz sciencewhiz changed the title Change install directory to ~/.wpilib or XDG_DATA_HOME/.wpilib Change install directory to ~/.wpilib or XDG_DATA_HOME/wpilib Jun 14, 2026

@Gold856 Gold856 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The settings.gradle in the tests don't match the ones in the VS Code extension. Is this intentional?

@ThadHouse

Copy link
Copy Markdown
Member Author

Correct. The tests are explicitly not for user use.

Gold856
Gold856 previously approved these changes Aug 14, 2026
@Gold856
Gold856 force-pushed the installtodotfolder branch from 6bd3358 to 632e963 Compare August 14, 2026 05:39
@PeterJohnson

Copy link
Copy Markdown
Member

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 632e963978

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread testing/java/settings.gradle
Comment thread src/main/java/org/wpilib/gradlerio/wpi/WPIExtension.java Outdated
checking the starts with ensures the test works when runs on windows
Comment thread testing/kotlin/settings.gradle
@PeterJohnson
PeterJohnson merged commit 487c43d into wpilibsuite:main Aug 28, 2026
6 checks passed
sciencewhiz added a commit to sciencewhiz/native-utils that referenced this pull request Aug 29, 2026
PeterJohnson pushed a commit to wpilibsuite/native-utils that referenced this pull request Aug 29, 2026
Sign up for free to 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.

5 participants