Skip to content

Conversation

@NotTheDr01ds
Copy link
Contributor

cc @Kissaki

Related #1609 and #1610

Hopefully this change (finally) gets us consistent results on all systems, since it now uses $nu.default-config-dir.

@Kissaki Thank you for finding and attempting to fix the issue! Clearly neither of our previous fixes works for the other (and that's still bugging me, so I may ping you on it again if I have ideas).

@fdncred fdncred merged commit 08bcbf8 into nushell:main Nov 2, 2024
2 checks passed
@fdncred
Copy link
Contributor

fdncred commented Nov 2, 2024

hopefully this is the last change with this. thanks for working on it @Kissaki and @NotTheDr01ds.

@NotTheDr01ds NotTheDr01ds deleted the yet-another-writing-modules-fix branch November 2, 2024 17:56
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