Skip to content
This repository was archived by the owner on Aug 15, 2023. It is now read-only.

fix: guardoni cli and electron build #371

Merged
merged 10 commits into from
Feb 9, 2022
Merged

Conversation

ascariandrea
Copy link
Collaborator

@ascariandrea ascariandrea commented Feb 7, 2022

Try the following commands:

yarn
yarn clean
yarn guardoni build
yarn guardoni pkg
yarn guardoni dist:all

This will produce the compiled code, the cli executables and the electron apps for the "development" environment (look at the guardoni/.env.development for the env vars loaded by default.

Run an instance of the yttrex backend with yarn backend watch

Then, to test the list command in the cli executable run:

DEBUG=@trex*,guardoni* ./guardoni/dist/guardoni-cli-2.2.0-$platform list --backend http://localhost:9000/api

# Output

# List succeeded: Experiments List


# Output values:

# experiments: []

You can test also other commands (have a look at the guardoni/README.md to see how to run them)

@ascariandrea ascariandrea force-pushed the fix/guardoni-fixes-for-demo branch from 13e208b to a2c6914 Compare February 7, 2022 15:13
@ascariandrea ascariandrea force-pushed the fix/guardoni-fixes-for-demo branch from a2c6914 to 88097a3 Compare February 8, 2022 09:03
@vecna
Copy link
Member

vecna commented Feb 9, 2022

should I merge it, in the meanwhile?
which are the still open bugs?

@ascariandrea
Copy link
Collaborator Author

I've opened #379 and #380 as next steps, so I can merge this.

@ascariandrea ascariandrea merged commit 8bd6339 into daily Feb 9, 2022
@ascariandrea ascariandrea deleted the fix/guardoni-fixes-for-demo branch February 9, 2022 14:49
ascariandrea added a commit that referenced this pull request Feb 16, 2022
chore(guardoni): dist-all shell script (#324)
chore(ycai): remove 'moment' dep (#356)
chore(guardoni): removed moment dep (#355)
fix(ycai): return total contributions in get creator stats api (#358)
chore(workspace): tsc diagnostics report (#357)
chore(yttrex): removed outdated create-react-class dep (#360)
chore(workspace): deps upgrade and "chai", "mocha", "immutability-hel…
chore(shared): fix DOTENV_CONFIG_PATH resolution in webpack config (#359
fix: guardoni cli and electron build (#371)
fix(guardoni): removed unused imports from cli file
chore(workspace): fix last version bump 2.2.2
ci: added 'preReleaseId' option to beta release
feat(backend): implemented originalHref into experiment CSV #384 (#385)
fix: guardoni electron dmg build and default paths (#387)
feat(ycai): gem collection (#345)
fix(backend): memory safer function for experiment extension (#393)
fix(ycai): gem card icons center alignment
fix(tktrex): tktrex extension data-donation size check (#373)
chore: pass env vars to ycai build on after release bump hook
fix: webpack extension outdir path
ci(ycai): release with proper env when merged in master [skip ci]
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants