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

PR #3855 reopened #3897

Merged
merged 2 commits into from
Jan 15, 2025
Merged

Conversation

brooksmtownsend
Copy link
Member

@brooksmtownsend brooksmtownsend commented Dec 31, 2024

Signed-off-by: Abhinav Prakash abhinav.prakash319@gmail.com

  • fix::> Proper exposure of path.rs
  • fix::> Proper logic of Paths resolution
  • fix::> replaced all std::env::current_dir() references in wash-cli
  • fix::> Changed the impl acc to discussion
  • fix::> publically exposed the config-path
  • fix::> Cleanupgit add crates/wash-cli/src/cmd/dev/mod.rs crates/wash-cli/src/cmd/mod.rs crates/wash-cli/src/cmd/wit/build.rs crates/wash-cli/src/cmd/wit/deps.rs crates/wash-lib/src/parser/mod.rs
  • fix::> Cleanupgit add crates/wash-cli/src/cmd/config/mod.rs
  • chore(wash-cli): remove unused import comment

Feature or Problem

It appears I am not a git wizard, and there's something up that closes PR #3855. These are all of the original commits from there by @PsychoPunkSage, who did a great job here, I just had to rebase the branch due to a conflict.

Related Issues

Originally #3855
Fixes #3800

Release Information

Consumer Impact

Testing

Unit Test(s)

Acceptance or Integration

Manual Verification

@brooksmtownsend brooksmtownsend requested a review from a team as a code owner December 31, 2024 14:52
@brooksmtownsend brooksmtownsend changed the title PR #3855 thank you git PR #3855 reopened Dec 31, 2024
@brooksmtownsend
Copy link
Member Author

Clippy issue will be fixed after #3898 merges, I will rebase

@lachieh
Copy link
Contributor

lachieh commented Dec 31, 2024

Looks like you'll have to correct the conventional commit formatting for these commits too

Copy link
Contributor

@thomastaylor312 thomastaylor312 left a comment

Choose a reason for hiding this comment

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

Good to go pending conventional commit fix

@brooksmtownsend brooksmtownsend force-pushed the PsychoPunkSage/main branch 2 times, most recently from 19228a7 to 43704c7 Compare December 31, 2024 17:15
…guration

Signed-off-by: Abhinav Prakash <abhinav.prakash319@gmail.com>
Signed-off-by: Brooks Townsend <brooksmtownsend@gmail.com>

fix: Proper logic of Paths resolution

Signed-off-by: Abhinav Prakash <abhinav.prakash319@gmail.com>
Signed-off-by: Brooks Townsend <brooksmtownsend@gmail.com>

fix: replaced all std::env::current_dir() references in wash-cli

Signed-off-by: Abhinav Prakash <abhinav.prakash319@gmail.com>
Signed-off-by: Brooks Townsend <brooksmtownsend@gmail.com>

fix: Changed the impl acc to discussion

Signed-off-by: Abhinav Prakash <abhinav.prakash319@gmail.com>
Signed-off-by: Brooks Townsend <brooksmtownsend@gmail.com>

fix: publically exposed the config-path

Signed-off-by: Abhinav Prakash <abhinav.prakash319@gmail.com>
Signed-off-by: Brooks Townsend <brooksmtownsend@gmail.com>

fix: Cleanupgit add crates/wash-cli/src/cmd/dev/mod.rs crates/wash-cli/src/cmd/mod.rs crates/wash-cli/src/cmd/wit/build.rs crates/wash-cli/src/cmd/wit/deps.rs crates/wash-lib/src/parser/mod.rs

Signed-off-by: Abhinav Prakash <abhinav.prakash319@gmail.com>
Signed-off-by: Brooks Townsend <brooksmtownsend@gmail.com>

fix: Cleanupgit add crates/wash-cli/src/cmd/config/mod.rs

Signed-off-by: Abhinav Prakash <abhinav.prakash319@gmail.com>
Signed-off-by: Brooks Townsend <brooksmtownsend@gmail.com>
Signed-off-by: Brooks Townsend <brooksmtownsend@gmail.com>
@brooksmtownsend brooksmtownsend added this pull request to the merge queue Jan 15, 2025
Merged via the queue into wasmCloud:main with commit 8827d0d Jan 15, 2025
106 checks passed
@brooksmtownsend brooksmtownsend deleted the PsychoPunkSage/main branch January 15, 2025 18:05
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.

[FEATURE] Rework current dir usage in wash-lib helpers
5 participants