Skip to content
This repository has been archived by the owner on Sep 13, 2023. It is now read-only.

Add setting to do deep requirement inspection #606

Merged
merged 2 commits into from
Feb 14, 2023

Conversation

mike0sv
Copy link
Contributor

@mike0sv mike0sv commented Feb 13, 2023

Always do deep inspections for local code
closes #548

Always do deep inspections for local code
closes #548
@mike0sv mike0sv requested a review from a team as a code owner February 13, 2023 14:15
@mike0sv mike0sv self-assigned this Feb 13, 2023
@mike0sv mike0sv temporarily deployed to internal February 13, 2023 14:15 — with GitHub Actions Inactive
@codecov
Copy link

codecov bot commented Feb 13, 2023

Codecov Report

Base: 86.07% // Head: 86.07% // Increases project coverage by +0.00% 🎉

Coverage data is based on head (c63aa31) compared to base (0c6f03c).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #606   +/-   ##
=======================================
  Coverage   86.07%   86.07%           
=======================================
  Files         107      107           
  Lines        9680     9682    +2     
=======================================
+ Hits         8332     8334    +2     
  Misses       1348     1348           
Impacted Files Coverage Δ
mlem/config.py 95.57% <100.00%> (+0.03%) ⬆️
mlem/utils/module.py 91.55% <100.00%> (+0.02%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@mike0sv mike0sv temporarily deployed to internal February 14, 2023 14:08 — with GitHub Actions Inactive
@mike0sv mike0sv temporarily deployed to internal February 14, 2023 16:40 — with GitHub Actions Inactive
@mike0sv mike0sv merged commit f51fbe2 into main Feb 14, 2023
@mike0sv mike0sv deleted the fix/false-neg-requirements branch February 14, 2023 18:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unexpected requirements parsing during save
2 participants