Skip to content

change delete return type#1134

Merged
Affie merged 1 commit intodevelopfrom
maint/verbs/delete
Jun 10, 2025
Merged

change delete return type#1134
Affie merged 1 commit intodevelopfrom
maint/verbs/delete

Conversation

@Affie
Copy link
Copy Markdown
Member

@Affie Affie commented Jun 10, 2025

delete should now return number of nodes deleted

Comment thread src/Deprecated.jl
Comment on lines +6 to +7
@deprecate addBlob!(store::AbstractBlobStore, blobId::UUID, data, ::String) addBlob!(store, blobId, data)
@deprecate addBlob!(store::AbstractBlobStore{T}, data::T, ::String) where {T} addBlob!(store, uuid4(), data)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[JuliaFormatter] reported by reviewdog 🐶

Suggested change
@deprecate addBlob!(store::AbstractBlobStore, blobId::UUID, data, ::String) addBlob!(store, blobId, data)
@deprecate addBlob!(store::AbstractBlobStore{T}, data::T, ::String) where {T} addBlob!(store, uuid4(), data)
@deprecate addBlob!(store::AbstractBlobStore, blobId::UUID, data, ::String) addBlob!(
store,
blobId,
data,
)
@deprecate addBlob!(store::AbstractBlobStore{T}, data::T, ::String) where {T} addBlob!(
store,
uuid4(),
data,
)

Comment thread src/Deprecated.jl
@@ -62,8 +70,3 @@ DFGSummary(args) = error("DFGSummary is deprecated")

@deprecate lsfWho(dfg::AbstractDFG, type::Symbol) lsf(dfg, getfield(Main, type))

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[JuliaFormatter] reported by reviewdog 🐶

Suggested change


deleteAgentMetadata!(dfg::AbstractDFG, key::Symbol) = pop!(dfg.agent.metadata, key)
deleteGraphMetadata!(dfg::AbstractDFG, key::Symbol) = pop!(dfg.graphMetadata, key)
function deleteAgentMetadata!(dfg::AbstractDFG, key::Symbol)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[JuliaFormatter] reported by reviewdog 🐶

Suggested change
function deleteAgentMetadata!(dfg::AbstractDFG, key::Symbol)
function deleteAgentMetadata!(dfg::AbstractDFG, key::Symbol)

@codecov
Copy link
Copy Markdown

codecov bot commented Jun 10, 2025

Codecov Report

Attention: Patch coverage is 73.17073% with 11 lines in your changes missing coverage. Please review.

Project coverage is 75.57%. Comparing base (eff25af) to head (2f82a96).
Report is 1 commits behind head on develop.

Files with missing lines Patch % Lines
src/DataBlobs/services/BlobStores.jl 50.00% 4 Missing ⚠️
src/DataBlobs/services/HelpersDataWrapEntryBlob.jl 50.00% 4 Missing ⚠️
src/DataBlobs/services/BlobEntry.jl 50.00% 2 Missing ⚠️
src/GraphsDFG/services/GraphsDFG.jl 75.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1134      +/-   ##
===========================================
+ Coverage    75.49%   75.57%   +0.08%     
===========================================
  Files           28       28              
  Lines         2387     2391       +4     
===========================================
+ Hits          1802     1807       +5     
+ Misses         585      584       -1     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Affie Affie merged commit 7b98ea2 into develop Jun 10, 2025
5 of 6 checks passed
@Affie Affie deleted the maint/verbs/delete branch June 10, 2025 03:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant