Skip to content

Conversation

@Kavindu-Dodan
Copy link
Contributor

@Kavindu-Dodan Kavindu-Dodan commented May 9, 2024

This PR

  • Handle FlagDisabledErrorCode and include a specific error message along with general error code
  • Improve error message for targeting rule validation failure

@Kavindu-Dodan Kavindu-Dodan requested a review from a team May 9, 2024 16:47
@netlify
Copy link

netlify bot commented May 9, 2024

Deploy Preview for polite-licorice-3db33c canceled.

Name Link
🔨 Latest commit 738a69a
🔍 Latest deploy log https://app.netlify.com/sites/polite-licorice-3db33c/deploys/663d2e6441791c00088eb38d

…ing rule

Signed-off-by: Kavindu Dodanduwa <kavindudodanduwa@gmail.com>
@Kavindu-Dodan Kavindu-Dodan force-pushed the feat/improve-ofrep-error-handling branch from 17da47d to 48c7217 Compare May 9, 2024 17:43
Signed-off-by: Kavindu Dodanduwa <kavindudodanduwa@gmail.com>
@codecov
Copy link

codecov bot commented May 9, 2024

Codecov Report

Attention: Patch coverage is 80.00000% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 77.31%. Comparing base (1c530ab) to head (738a69a).
Report is 59 commits behind head on main.

Files Patch % Lines
core/pkg/evaluator/json.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1306      +/-   ##
==========================================
+ Coverage   73.69%   77.31%   +3.61%     
==========================================
  Files          32       20      -12     
  Lines        3140     1622    -1518     
==========================================
- Hits         2314     1254    -1060     
+ Misses        717      286     -431     
+ Partials      109       82      -27     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: Kavindu Dodanduwa <kavindudodanduwa@gmail.com>
@Kavindu-Dodan Kavindu-Dodan merged commit 39ae4fe into open-feature:main May 10, 2024
@Kavindu-Dodan Kavindu-Dodan deleted the feat/improve-ofrep-error-handling branch May 10, 2024 14:35
@github-actions github-actions bot mentioned this pull request May 9, 2024
Kavindu-Dodan pushed a commit that referenced this pull request May 10, 2024
🤖 I have created a release *beep* *boop*
---


<details><summary>flagd: 0.10.2</summary>

##
[0.10.2](flagd/v0.10.1...flagd/v0.10.2)
(2024-05-10)


### ✨ New Features

* Create interface for eval events.
([#1288](#1288))
([9714215](9714215))


### 🧹 Chore

* bump go deps to latest
([#1307](#1307))
([004ad08](004ad08))
</details>

<details><summary>flagd-proxy: 0.6.2</summary>

##
[0.6.2](flagd-proxy/v0.6.1...flagd-proxy/v0.6.2)
(2024-05-10)


### 🧹 Chore

* bump go deps to latest
([#1307](#1307))
([004ad08](004ad08))
</details>

<details><summary>core: 0.9.2</summary>

##
[0.9.2](core/v0.9.1...core/v0.9.2)
(2024-05-10)


### ✨ New Features

* improve error log and add flag disabled handling for ofrep
([#1306](#1306))
([39ae4fe](39ae4fe))


### 🧹 Chore

* bump go deps to latest
([#1307](#1307))
([004ad08](004ad08))
</details>

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Signed-off-by: OpenFeature Bot <109696520+openfeaturebot@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

3 participants