Skip to content

Conversation

julesvanrie
Copy link
Contributor

@julesvanrie julesvanrie commented Oct 14, 2025

The locale problem also happens on some Ubuntu installations.

The solution currently sits in the Windows Update partial, but it makes more sense to include it in a partial that caters for both Windows and Ubuntu.

The start of the CLI tools section is a good place to do this:

  • It fits better with the setup actions we do in that section.
  • It is still early enough: the problem only gets triggered after dotfiles.

This commit will automatically move this in the right place. No need to change the build.rb. The next rebuild of the data-setup will then also include these changes. DA and DE setups do not rely on these partials.

Also corrected some typos in the FR partial.

The locale problem also happens on some Ubuntu installations.

The solution currently sits in the Windows Update partial, but it makes
more sense to include it in a partial that caters for both Windows and
Ubuntu.

The start of the CLI tools section is a good place to do this:
- It fits  better with the setup actions we do in that section.
- It is still early enough: the problem only gets triggered after dotfiles.

This commit will automatically move this in the right place. No need to
change the `build.rb`. The next rebuild of the data-setup will then also
include these changes. DA and DE setups do not rely on these partials.

Also corrected some typos in the FR partial.
@julesvanrie julesvanrie requested a review from ajdubovoy October 14, 2025 19:01
Copy link
Contributor

@ajdubovoy ajdubovoy left a comment

Choose a reason for hiding this comment

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

Makes perfect sense to me, thanks!

@ajdubovoy ajdubovoy requested a review from Eschults October 16, 2025 01:04
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.

2 participants