Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

apps: add namadac utils default-base-dir #1491

Merged
merged 2 commits into from
Jun 14, 2023
Merged

apps: add namadac utils default-base-dir #1491

merged 2 commits into from
Jun 14, 2023

Conversation

juped
Copy link
Member

@juped juped commented Jun 1, 2023

Add a utility command to print the default base directory that would be
used by the Namada command line if a base directory were not provided
on the command line or in the environment.

juped added a commit that referenced this pull request Jun 1, 2023
Add a utility command to print the default base directory that would be
used by the Namada command line if a base directory were not provided
on the command line or in the environment.
juped added a commit that referenced this pull request Jun 1, 2023
println!(
"{}",
get_default_namada_folder().to_str().expect(
"expected a default namada folder to be possible to determine"
Copy link

@gijswijs gijswijs Jun 2, 2023

Choose a reason for hiding this comment

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

<pedantic>Error message is a bit cryptic</pedantic>

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah +1 I'd like to see something else

Copy link

@gijswijs gijswijs left a comment

Choose a reason for hiding this comment

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

In theory we should have a unit test even for small features like this.

Fraccaman added a commit that referenced this pull request Jun 9, 2023
* origin/ray/utils-base-dir:
  changelog: add #1491
  apps: add namadac utils default-base-dir
@Fraccaman Fraccaman mentioned this pull request Jun 9, 2023
Fraccaman added a commit that referenced this pull request Jun 14, 2023
* origin/ray/utils-base-dir:
  changelog: add #1491
  apps: add namadac utils default-base-dir
Fraccaman pushed a commit that referenced this pull request Jun 14, 2023
@Fraccaman Fraccaman merged commit f40c1bc into main Jun 14, 2023
@Fraccaman Fraccaman deleted the ray/utils-base-dir branch June 14, 2023 14:41
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.

4 participants