Skip to content

fix: fixing silent lint failures#550

Merged
Kavindu-Dodan merged 1 commit intoopen-feature:mainfrom
Kavindu-Dodan:fix/enable-lint-failures
Mar 23, 2023
Merged

fix: fixing silent lint failures#550
Kavindu-Dodan merged 1 commit intoopen-feature:mainfrom
Kavindu-Dodan:fix/enable-lint-failures

Conversation

@Kavindu-Dodan
Copy link
Copy Markdown
Contributor

@Kavindu-Dodan Kavindu-Dodan commented Mar 22, 2023

This PR

Fix lint check & adds lint fixes for existing lint failures (reason for 16 file changes)

See this example [1], linting was failing silently as we rely on a foreach loop.

Solution credits - https://stackoverflow.com/a/22214424/3197055

[1] - https://github.com/open-feature/flagd/actions/runs/4471193065/jobs/7855746534#step:5:640

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 22, 2023

Codecov Report

Merging #550 (0781726) into main (41799f6) will increase coverage by 0.04%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##             main     #550      +/-   ##
==========================================
+ Coverage   66.66%   66.71%   +0.04%     
==========================================
  Files          19       19              
  Lines        2133     2127       -6     
==========================================
- Hits         1422     1419       -3     
+ Misses        651      648       -3     
  Partials       60       60              
Impacted Files Coverage Δ
core/pkg/runtime/runtime.go 0.00% <0.00%> (ø)
core/pkg/sync-store/sync_store.go 83.51% <ø> (ø)
core/pkg/sync/grpc/credentials/builder.go 100.00% <ø> (ø)
core/pkg/sync/grpc/grpc_sync.go 90.97% <ø> (ø)
core/pkg/sync/http/http_sync.go 43.29% <0.00%> (ø)
core/pkg/eval/json_evaluator.go 87.72% <100.00%> (-0.17%) ⬇️
...ore/pkg/service/flag-evaluation/connect_service.go 63.63% <100.00%> (ø)
core/pkg/service/flag-evaluation/flag_evaluator.go 61.81% <100.00%> (ø)
core/pkg/service/middleware/http_metrics.go 76.25% <100.00%> (ø)
core/pkg/sync/file/filepath_sync.go 54.54% <100.00%> (ø)
... and 1 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Copy Markdown
Member

@thisthat thisthat left a comment

Choose a reason for hiding this comment

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

Nice catch 🙌

Signed-off-by: Kavindu Dodanduwa <kavindudodanduwa@gmail.com>
@toddbaert toddbaert force-pushed the fix/enable-lint-failures branch from 31fa27b to 0781726 Compare March 23, 2023 12:16
@Kavindu-Dodan Kavindu-Dodan merged commit 30c8022 into open-feature:main Mar 23, 2023
@github-actions github-actions Bot mentioned this pull request Mar 23, 2023
toddbaert added a commit that referenced this pull request Mar 30, 2023
🤖 I have created a release *beep* *boop*
---


<details><summary>flagd: 0.5.0</summary>

##
[0.5.0](flagd/v0.4.5...flagd/v0.5.0)
(2023-03-30)


### ⚠ BREAKING CHANGES

* unify sources configuration handling
([#560](#560))

### 🐛 Bug Fixes

* benchmark pipeline
([#538](#538))
([62cc0fc](62cc0fc))
* **deps:** update module github.com/open-feature/flagd/core to v0.4.5
([#552](#552))
([41799f6](41799f6))


### 🧹 Chore

* refactor configuration handling for startup
([#551](#551))
([8dfbde5](8dfbde5))
</details>

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

##
[0.2.0](flagd-proxy-v0.1.2...flagd-proxy/v0.2.0)
(2023-03-30)


### ⚠ BREAKING CHANGES

* rename `kube-flagd-proxy` to `flagd-proxy`
([#576](#576))

### ✨ New Features

* rename `kube-flagd-proxy` to `flagd-proxy`
([#576](#576))
([223de99](223de99))
</details>

<details><summary>core: 0.5.0</summary>

##
[0.5.0](core/v0.4.5...core/v0.5.0)
(2023-03-30)


### ⚠ BREAKING CHANGES

* rename `kube-flagd-proxy` to `flagd-proxy`
([#576](#576))
* unify sources configuration handling
([#560](#560))

### 🧹 Chore

* move credential builder for grpc sync into seperate component
([#536](#536))
([7314fee](7314fee))
* refactor configuration handling for startup
([#551](#551))
([8dfbde5](8dfbde5))
* refactor middleware setup in server
([#554](#554))
([01016c7](01016c7))
* refactor service configuration objects
([#545](#545))
([c7b29ed](c7b29ed)),
closes [#524](#524)
* unify sources configuration handling
([#560](#560))
([7f4888a](7f4888a))


### 🐛 Bug Fixes

* **deps:** update module google.golang.org/grpc to v1.54.0
([#548](#548))
([99ba5ec](99ba5ec))
* **deps:** update module sigs.k8s.io/controller-runtime to v0.14.6
([#572](#572))
([bed9458](bed9458))
* fixing silent lint failures
([#550](#550))
([30c8022](30c8022))
* nil pointer fix + export constructors
([#555](#555))
([78adb81](78adb81))


### ✨ New Features

* expose Impression metric
([#556](#556))
([77e0a33](77e0a33))
* Introduce kube-proxy-metrics
([#558](#558))
([ad0baeb](ad0baeb))
* rename `kube-flagd-proxy` to `flagd-proxy`
([#576](#576))
([223de99](223de99))
* refactor core module into multiple packages
([#530](#530))
([9d68d0b](9d68d0b))
</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: Todd Baert <toddbaert@gmail.com>
Co-authored-by: Todd Baert <toddbaert@gmail.com>
@github-actions github-actions Bot mentioned this pull request Dec 2, 2023
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