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: allow evaluating in CLI with more verbosity #300

Merged
merged 1 commit into from
Apr 30, 2024
Merged

Conversation

fahad19
Copy link
Member

@fahad19 fahad19 commented Apr 30, 2024

What's done

SDK evaluation reason

SDK evaluation reason expanded with a new reason no_match.

This reason is used when evaluation logic cannot not match with any specific rule against given context.

Verbose evaluation

We allow evaluating features in CLI already: https://featurevisor.com/docs/cli/#evaluate

A new --verbose option has been introduced that will print SDK logs for debugging even better, if needs arise.

@fahad19 fahad19 self-assigned this Apr 30, 2024
@@ -87,6 +87,7 @@ export type DatafileFetchHandler = (datafileUrl: string) => Promise<DatafileCont
export enum EvaluationReason {
Copy link
Member Author

Choose a reason for hiding this comment

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

if you are an SDK developer in other languages, this file change will be of your interest.

@fahad19 fahad19 merged commit 8c3a096 into main Apr 30, 2024
1 check passed
@fahad19 fahad19 deleted the cli-evaluate-logs branch April 30, 2024 18:06
fahad19 added a commit that referenced this pull request May 25, 2024
* main: (48 commits)
  v1.22.0
  feat: show project info in CLI (#304)
  docs: deprecation (#302)
  v1.21.0
  feat: make test runner fast by default (#301)
  v1.20.0
  feat: allow evaluating in CLI with more verbosity (#300)
  docs: glossary (#291)
  v1.19.0
  feat: find duplicate segments with authors info (#299)
  v1.18.1
  fix: search features by any case in generated site (#298)
  v1.18.0
  feat: assess traffic distribution via CLI (#297)
  docs: updates
  v1.17.0
  feat: evaluate features in CLI (#295)
  v1.16.0
  feat: Added export of emitter module (#294)
  v1.15.0
  ...
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

1 participant