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

chore: move washboard to its own directory #764

Merged
merged 1 commit into from
Oct 31, 2023

Conversation

connorsmith256
Copy link
Contributor

Feature or Problem

This is the minimal amount of work to split the washboard directory out of wash into its own top-level directory

The washboard doesn't have any CI yet

@lachieh should I commit changes to dist/?

Testing

Manual Verification

  • built the washboard: cd washboard && make build-ui && cd -
  • built wash: cd wash && cargo build
  • ran wash: ./target/debug/wash up --nats-websocket-port 4001
  • ran the washboard: ./target/debug/wash ui --experimental

@connorsmith256 connorsmith256 requested review from a team as code owners October 19, 2023 03:40
@lachieh
Copy link
Contributor

lachieh commented Oct 19, 2023

@lachieh should I commit changes to dist/?

Yes please, until we have a release process

rvolosatovs
rvolosatovs previously approved these changes Oct 19, 2023
vados-cosmonic pushed a commit to vados-cosmonic/wasmCloud that referenced this pull request Oct 19, 2023
@connorsmith256 connorsmith256 marked this pull request as draft October 19, 2023 16:07
@connorsmith256
Copy link
Contributor Author

On second thought, I'll leave this in draft until we've archived the wash repo. I don't want to make it more difficult to cherry-pick changes from there

washboard-ui/Makefile Outdated Show resolved Hide resolved
rvolosatovs
rvolosatovs previously approved these changes Oct 31, 2023
Copy link
Member

@rvolosatovs rvolosatovs left a comment

Choose a reason for hiding this comment

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

I'm very surprised that using the relative path to parent directory for the washboard worked here

Copy link
Member

@brooksmtownsend brooksmtownsend left a comment

Choose a reason for hiding this comment

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

Looks good w/ Lachlan's request

Signed-off-by: Connor Smith <connor.smith.256@gmail.com>
@connorsmith256 connorsmith256 enabled auto-merge (rebase) October 31, 2023 16:49
@connorsmith256 connorsmith256 merged commit b936abf into wasmCloud:main Oct 31, 2023
30 checks passed
@rvolosatovs
Copy link
Member

Looks like you forgot to update the paths triggers for wash CI, which builds UI

@connorsmith256 connorsmith256 deleted the chore/split-washboard branch October 31, 2023 17:57
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.

None yet

4 participants