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

refactor: remove unused struct field #458

Merged
merged 4 commits into from
Mar 3, 2023
Merged

refactor: remove unused struct field #458

merged 4 commits into from
Mar 3, 2023

Conversation

james-milligan
Copy link
Contributor

This PR

  • removes unused struct field in filepath sync

Related Issues

Notes

Follow-up Tasks

How to test

Signed-off-by: James Milligan <[email protected]>
@codecov
Copy link

codecov bot commented Mar 2, 2023

Codecov Report

Merging #458 (8e15671) into main (b4ee495) will not change coverage.
The diff coverage is 60.00%.

❗ Current head 8e15671 differs from pull request most recent head 7e8fb38. Consider uploading reports for the commit 7e8fb38 to get more accurate results

@@           Coverage Diff           @@
##             main     #458   +/-   ##
=======================================
  Coverage   62.26%   62.26%           
=======================================
  Files          13       13           
  Lines        1659     1659           
=======================================
  Hits         1033     1033           
  Misses        563      563           
  Partials       63       63           
Impacted Files Coverage Δ
pkg/sync/kubernetes/kubernetes_sync.go 36.36% <50.00%> (ø)
pkg/sync/file/filepath_sync.go 54.54% <100.00%> (ø)

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

@Kavindu-Dodan
Copy link
Contributor

Can you also check K8s sync - https://github.com/open-feature/flagd/blob/main/pkg/sync/kubernetes/kubernetes_sync.go#L33

Signed-off-by: James Milligan <[email protected]>
@beeme1mr beeme1mr self-requested a review March 2, 2023 19:29
Signed-off-by: Kavindu Dodanduwa <[email protected]>
@Kavindu-Dodan Kavindu-Dodan merged commit a04c0b8 into open-feature:main Mar 3, 2023
beeme1mr pushed a commit that referenced this pull request Mar 7, 2023
🤖 I have created a release *beep* *boop*
---


##
[0.4.1](v0.4.0...v0.4.1)
(2023-03-07)


### 🔄 Refactoring

* remove unused struct field
([#458](#458))
([a04c0b8](a04c0b8))


### 🧹 Chore

* **deps:** update sigstore/cosign-installer digest to bd2d118
([#471](#471))
([ee90f48](ee90f48))


### 🐛 Bug Fixes

* **deps:** update module
github.com/open-feature/go-sdk-contrib/providers/flagd to v0.1.10
([#459](#459))
([cbdf9b0](cbdf9b0))
* **deps:** update module golang.org/x/net to v0.8.0
([#468](#468))
([10d5f2c](10d5f2c))
* fix broken image signing
([#461](#461))
([05bb51c](05bb51c))
* fixing image delimeter
([#463](#463))
([b4ee495](b4ee495))
* security issues
([#464](#464))
([7f1e759](7f1e759))
* set readiness once only
([#465](#465))
([41a888d](41a888d))

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

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@james-milligan james-milligan deleted the remove-unused-struct-field branch March 23, 2023 10:58
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