Browse Source

chore: remove outdated todos

Evan You 7 years ago
parent
commit
8de1c484ff

+ 0 - 1
packages/core/src/createRenderer.ts

@@ -149,7 +149,6 @@ export function createRenderer(options: RendererOptions) {
     newNode: RenderNode | RenderFragment,
     refNode: RenderNode | RenderFragment | null
   ) {
-    // TODO special case for Fragment
     if (refNode === null) {
       appendChild(container, newNode)
     } else {

+ 0 - 3
packages/observer/__tests__/autorun.spec.ts

@@ -323,8 +323,6 @@ describe('observer/autorun', () => {
 
   it('should allow explicitly recursive raw function loops', () => {
     const counter = observable({ num: 0 })
-
-    // TODO: this should be changed to autorun loops, can it be done?
     const numSpy = jest.fn(() => {
       counter.num++
       if (counter.num < 10) {
@@ -332,7 +330,6 @@ describe('observer/autorun', () => {
       }
     })
     autorun(numSpy)
-
     expect(counter.num).toEqual(10)
     expect(numSpy).toHaveBeenCalledTimes(10)
   })