Skip to content

a stale observer must not update the display#175

Closed
Fil wants to merge 1 commit into
mainfrom
fil/stale-variable
Closed

a stale observer must not update the display#175
Fil wants to merge 1 commit into
mainfrom
fil/stale-variable

Conversation

@Fil

@Fil Fil commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

closes #174

todo:

  • add test
  • type Observer? like so:
  interface Observer {
    pending?(): void;
    fulfilled?(value: unknown): void;
    rejected?(error: unknown): void;
  }

@Fil Fil requested a review from mbostock July 3, 2026 21:47
@Fil Fil marked this pull request as draft July 3, 2026 21:48
@mbostock

mbostock commented Jul 4, 2026

Copy link
Copy Markdown
Member

I don’t think this is the right fix, and I’m not sure that the linked issue is a bug. I’ll look more…

@Fil

Fil commented Jul 4, 2026

Copy link
Copy Markdown
Contributor Author

#174 (comment)

@Fil Fil closed this Jul 4, 2026
@Fil Fil deleted the fil/stale-variable branch July 4, 2026 07:59
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.

How to properly redefine a cell?

2 participants