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

feat(core): allow CLI command configurations #2972

Merged
merged 8 commits into from
Sep 28, 2019

Conversation

spkjp
Copy link
Contributor

@spkjp spkjp commented Sep 26, 2019

Summary

Allows changing the default CLI command configuration.

Checklist

  • Documentation (if necessary)
  • Tests (if necessary)
  • Ready to be merged

@codecov
Copy link

codecov bot commented Sep 26, 2019

Codecov Report

Merging #2972 into develop will increase coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #2972      +/-   ##
===========================================
+ Coverage    65.95%   65.96%   +<.01%     
===========================================
  Files          420      420              
  Lines        10014    10033      +19     
  Branches       508      509       +1     
===========================================
+ Hits          6605     6618      +13     
- Misses        3369     3374       +5     
- Partials        40       41       +1
Impacted Files Coverage Δ
packages/core/src/utils.ts 0% <0%> (ø) ⬆️
packages/core/src/commands/relay/run.ts 0% <0%> (ø) ⬆️
packages/core/src/commands/forger/run.ts 0% <0%> (ø) ⬆️
packages/core/src/commands/core/run.ts 0% <0%> (ø) ⬆️
packages/core/src/helpers/replay.ts 0% <0%> (ø) ⬆️
packages/core/src/helpers/snapshot.ts 0% <0%> (ø) ⬆️
...ckages/core-api/src/handlers/businesses/methods.ts 57.89% <0%> (+5.26%) ⬆️
...ages/core-api/src/handlers/bridgechains/methods.ts 57.89% <0%> (+5.26%) ⬆️
...kages/core-api/src/handlers/bridgechains/routes.ts 100% <0%> (+62.5%) ⬆️
...ackages/core-api/src/handlers/businesses/routes.ts 100% <0%> (+66.66%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6c6c96c...81f0954. Read the comment docs.

faustbrian
faustbrian previously approved these changes Sep 26, 2019
faustbrian
faustbrian previously approved these changes Sep 27, 2019
@spkjp spkjp force-pushed the feat/core/cli-command-configuration branch from 5061df7 to 81f0954 Compare September 28, 2019 01:15
@spkjp spkjp merged commit b6bc4b4 into develop Sep 28, 2019
@ghost ghost deleted the feat/core/cli-command-configuration branch September 28, 2019 02:03
vasild added a commit that referenced this pull request Oct 1, 2019
…ts-nonce

* ArkEcosystem/core/develop:
  fix(deps): update dependency @hapi/boom to v8 (#2988)
  feat(crypto): allow passing height to `configManager.isNewMilestone` (#3001)
  ci: update branch patterns to new format
  refactor: remove asset migration heuristic (#2999)
  chore(deps): update dependency typesync to ^0.6.0 (#2981)
  refactor(core): use default heap size regardless of available memory (#2998)
  chore(deps): update dependency cross-env to v6 (#2985)
  chore(deps): update dependency rimraf to v3 (#2987)
  refactor: core-magistrate-crypto export namespaces (#2980)
  feat: multiSignWif (#2979)
  fix(core-snapshots): export/import transactions' type_group (#2996)
  chore(deps): update dependency @types/newrelic to v5 (#2984)
  fix(deps): update dependency node-forge to ^0.9.0 (#2983)
  feat(core-api): find htlc unlock transactions (#2976)
  chore(deps): update dependency del-cli to v3 (#2986)
  fix(deps): update dependency cli-progress to v3 (#2990)
  fix(deps): update dependency winston-daily-rotate-file to v4 (#2992)
  fix(core-api): missing orderBy (#2974)
  feat(core): allow CLI command configurations (#2972)
  refactor(core-magistrate):  Split core-marketplace into core-magistrate-crypto and core-magistrate-transactions (#2967)
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.

2 participants