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: add fetch functions to Repo module #322

Merged
merged 1 commit into from
Jul 25, 2023

Conversation

noaccOS
Copy link
Contributor

@noaccOS noaccOS commented Jul 11, 2023

they work like get functions but always return a tuple {:ok, item} or {:error, :not_found}

backend/lib/edgehog/repo.ex Outdated Show resolved Hide resolved
they work like get functions but always return a tuple {:ok, item} or
{:error, :not_found}

Signed-off-by: Francesco Noacco <francesco.noacco@secomind.com>
@coveralls
Copy link

Pull Request Test Coverage Report for Build 8385b922ef3daea6d9973663cecfc82b6b1dc59f-PR-322

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 8 unchanged lines in 3 files lost coverage.
  • Overall coverage increased (+0.04%) to 82.272%

Files with Coverage Reduction New Missed Lines %
lib/edgehog/astarte.ex 1 98.36%
lib/edgehog/repo.ex 2 88.24%
lib/edgehog/update_campaigns.ex 5 92.0%
Totals Coverage Status
Change from base Build 0f12af445a03d498ae6f705b87add609b8d39465: 0.04%
Covered Lines: 1137
Relevant Lines: 1382

💛 - Coveralls

@rbino rbino merged commit f573ac6 into edgehog-device-manager:main Jul 25, 2023
4 checks passed
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