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

added option to include errors in tweek-api values result #1132

Merged
merged 10 commits into from Apr 7, 2019

Conversation

Projects
None yet
2 participants
@nataly87s
Copy link
Contributor

commented Apr 2, 2019

No description provided.

@ghost ghost assigned nataly87s Apr 2, 2019

@ghost ghost added the in progress label Apr 2, 2019

@nataly87s nataly87s requested a review from Yshayy Apr 2, 2019

@nataly87s nataly87s changed the title added option to propagate errors from tweek-api added option to include errors in tweek-api values result Apr 3, 2019

@nataly87s nataly87s force-pushed the api-error-handling branch from f4d31d6 to 5cc7b38 Apr 7, 2019

@rosensj

rosensj approved these changes Apr 7, 2019

@nataly87s nataly87s merged commit 9abc93c into master Apr 7, 2019

11 of 14 checks passed

security/snyk - services/authoring/package.json (Soluto Public) 5 high severity vulnerable dependency paths
Details
license/snyk - services/api/Tweek.ApiService/Tweek.ApiService.csproj (Soluto Public) Test in progress
license/snyk - services/publishing/Tweek.Publishing.Service/Tweek.Publishing.Service.csproj (Soluto Public) Test in progress
ci/circleci: build Your tests passed on CircleCI!
Details
license/snyk - package.json (Soluto Public) No high severity issues
Details
license/snyk - services/api/Tweek.ApiService/Tweek.ApiService.csproj - netcoreapp2.1 (Soluto Public) No high severity issues
Details
license/snyk - services/authoring/package.json (Soluto Public) No high severity issues
Details
license/snyk - services/editor/package.json (Soluto Public) No high severity issues
Details
license/snyk - services/publishing/Tweek.Publishing.Service/Tweek.Publishing.Service.csproj - netcoreapp2.1 (Soluto Public) No high severity issues
Details
security/snyk - package.json (Soluto Public) No high severity issues
Details
security/snyk - services/api/Tweek.ApiService/Tweek.ApiService.csproj - netcoreapp2.1 (Soluto Public) No high severity issues
Details
security/snyk - services/editor/package.json (Soluto Public) No high severity issues
Details
security/snyk - services/publishing/Tweek.Publishing.Service/Tweek.Publishing.Service.csproj - netcoreapp2.1 (Soluto Public) No high severity issues
Details
tweek-all Build passed
Details

@nataly87s nataly87s deleted the api-error-handling branch Apr 7, 2019

@ghost ghost removed the in progress label Apr 7, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.