Skip to content

Conversation

@raju-opti
Copy link
Contributor

Summary

  • log an error when datafile fetch request returns a non success status code

Test plan

  • added test for the logging

Issues

  • FSSDK-9615

@coveralls
Copy link

coveralls commented Feb 23, 2024

Coverage Status

coverage: 90.373% (-0.007%) from 90.38%
when pulling 37ea6c5 on raju/fetch-log
into d5e6a05 on master.

@raju-opti
Copy link
Contributor Author

solves #784

Copy link
Contributor

@mikechu-optimizely mikechu-optimizely left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@mikechu-optimizely
Copy link
Contributor

Actually, please update the copyright date.

We probably need to devise an Actions workflow that looks for that on files included in PRs

@raju-opti raju-opti merged commit e938e3c into master Feb 23, 2024
@raju-opti raju-opti deleted the raju/fetch-log branch February 23, 2024 14:36
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.

4 participants