Skip to content

feat: allow running workflows using cli command - #903

Merged
dOrgJelli merged 30 commits into
prealpha-devfrom
nk/cli-workflow
Jun 23, 2022
Merged

feat: allow running workflows using cli command#903
dOrgJelli merged 30 commits into
prealpha-devfrom
nk/cli-workflow

Conversation

@Niraj-Kamdar

@Niraj-Kamdar Niraj-Kamdar commented Jun 8, 2022

Copy link
Copy Markdown
Contributor

Linked Issues

Changelog

  • Added support for running workflow files from w3 cli
  • Added support for basic tests by validating output

Next steps

  • Add option to create workflow project (post-alpha)

@Niraj-Kamdar
Niraj-Kamdar requested a review from dOrgJelli as a code owner June 8, 2022 16:40
@Niraj-Kamdar
Niraj-Kamdar marked this pull request as draft June 8, 2022 16:41
@cbrzn

cbrzn commented Jun 9, 2022

Copy link
Copy Markdown
Contributor

now that this is implemented, would it make sense to close PR #725 ?

Base automatically changed from nk/recipe-refactor to prealpha-dev June 9, 2022 15:09
@dOrgJelli

Copy link
Copy Markdown
Contributor

Yes @cbrzn, will close that PR

@Niraj-Kamdar
Niraj-Kamdar marked this pull request as ready for review June 10, 2022 16:23
Running Recipe just after starting test-env doesn't guarantee all the necessary services has started and since test-env-js comes with awaiter that makes surethe services has started, it's better to test it in jest.
@Niraj-Kamdar
Niraj-Kamdar requested review from cbrzn and krisbitney June 13, 2022 10:57
Comment thread .w3/infra/modules/base-docker-compose.yml Outdated
Comment thread packages/cli/lang/en.json Outdated
Comment thread packages/cli/lang/en.json Outdated
Comment thread packages/cli/lang/es.json Outdated
Comment thread packages/cli/lang/es.json Outdated
Comment thread packages/cli/lang/es.json Outdated
Comment thread packages/cli/src/commands/run.ts Outdated
Comment thread packages/templates/api/assemblyscript/workflows/config.ts Outdated
Comment thread packages/templates/api/assemblyscript/workflows/e2e.yaml Outdated
Comment thread packages/test-cases/cases/cli/wasm/run/workflows/config.ts Outdated
Comment thread packages/test-cases/cases/cli/wasm/run/workflows/e2e.json Outdated
Comment thread packages/test-cases/cases/cli/wasm/run/workflows/e2e.yaml Outdated
Niraj-Kamdar and others added 5 commits June 20, 2022 07:10
Co-authored-by: Cesar Brazon <cesarbrazon10@gmail.com>
Co-authored-by: Cesar Brazon <cesarbrazon10@gmail.com>
cbrzn
cbrzn previously approved these changes Jun 20, 2022

@cbrzn cbrzn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM 🍏

dOrgJelli
dOrgJelli previously approved these changes Jun 23, 2022

@dOrgJelli dOrgJelli left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🔥 🔥 🔥 🔥 🔥 🔥 🔥 🔥 🔥 🔥

@dOrgJelli
dOrgJelli dismissed stale reviews from cbrzn and themself via caceabf June 23, 2022 18:03
@dOrgJelli
dOrgJelli merged commit c5fb839 into prealpha-dev Jun 23, 2022
@dOrgJelli
dOrgJelli deleted the nk/cli-workflow branch April 10, 2023 16:53
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.

3 participants