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

fix: Follow-up fixes for #323 #338

Merged
merged 2 commits into from
Oct 4, 2020

Conversation

achimnol
Copy link
Member

@achimnol achimnol commented Oct 4, 2020

No description provided.

* fix: Correct null-checks for unchanged vluaes in update_vfolder_options()
@achimnol achimnol force-pushed the refactor/rename-clone-allowed-to-clonable branch from 15e8166 to 7e89b03 Compare October 4, 2020 05:28
@codecov
Copy link

codecov bot commented Oct 4, 2020

Codecov Report

Merging #338 into master will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #338   +/-   ##
=======================================
  Coverage   47.56%   47.56%           
=======================================
  Files          48       48           
  Lines        7491     7491           
=======================================
  Hits         3563     3563           
  Misses       3928     3928           
Impacted Files Coverage Δ
src/ai/backend/manager/models/vfolder.py 39.37% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6bd977e...007b259. Read the comment docs.

@achimnol achimnol merged commit abf105b into master Oct 4, 2020
@achimnol achimnol deleted the refactor/rename-clone-allowed-to-clonable branch October 4, 2020 05:36
achimnol added a commit that referenced this pull request Oct 5, 2020
…, #338)

* Both clonable and cloneable are considered valid English, but in the CS use cases,
  cloneable is more prevalent.
@achimnol achimnol added this to the 20.09 milestone Oct 5, 2020
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant