Skip to content

Conversation

@regexident
Copy link
Contributor

Salsa also supports tracked methods and tracked associated methods, neither of which produce equivalent user-observable structs, so arguably neither should track functions, and the generated structs don't really provide a usable API anyway.

@netlify
Copy link

netlify bot commented Jun 17, 2025

Deploy Preview for salsa-rs canceled.

Name Link
🔨 Latest commit d4d6fec
🔍 Latest deploy log https://app.netlify.com/projects/salsa-rs/deploys/68518d55d019c200081ea71f

@regexident regexident force-pushed the doc-hidden-fn-structs branch from 54c5ebd to d4d6fec Compare June 17, 2025 15:44
@codspeed-hq
Copy link

codspeed-hq bot commented Jun 17, 2025

CodSpeed Performance Report

Merging #917 will not alter performance

Comparing regexident:doc-hidden-fn-structs (d4d6fec) with master (87a730f)

Summary

✅ 12 untouched benchmarks

@MichaReiser MichaReiser requested a review from Veykril June 17, 2025 15:52
@Veykril Veykril added this pull request to the merge queue Jun 25, 2025
Merged via the queue into salsa-rs:master with commit 8d14c02 Jun 25, 2025
12 checks passed
@github-actions github-actions bot mentioned this pull request Jun 25, 2025
@regexident regexident deleted the doc-hidden-fn-structs branch June 25, 2025 15:10
@github-actions github-actions bot mentioned this pull request Jun 28, 2025
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.

3 participants