Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

removed references to json things #727

Merged

Conversation

K20shores
Copy link
Collaborator

Closes #675

  • Removed the no json dockerfile because it wasn't even testing anything and this is soon to be obsolete anyway

@K20shores K20shores linked an issue Feb 19, 2025 that may be closed by this pull request
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.48%. Comparing base (001a4be) to head (a077cff).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #727   +/-   ##
=======================================
  Coverage   94.48%   94.48%           
=======================================
  Files          64       64           
  Lines        4408     4408           
=======================================
  Hits         4165     4165           
  Misses        243      243           

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

@K20shores K20shores merged commit e08c4f7 into main Feb 19, 2025
26 checks passed
@K20shores K20shores deleted the 675-remove-references-to-nlohmann-json-in-docker-actions branch February 19, 2025 17:03
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove references to nlohmann json in docker actions
4 participants