Skip to content

Conversation

@StoneCypher
Copy link

On install, attempt to:

  • install git submodules automatically,
  • set up workspaces automatically,
  • and build once

Also, stub the standard build call to point to project build.

Currently untested due to #82

…tomatically, and build once on install. Also, stub the standard build call to point to project build. Currently untested due to KnodesCommunity#82
} );
const out = testHost.mapPagesToReflections( [ { title: 'HIDDEN', source: 'foo.md' } ] );
expect( out ).toHaveLength( 1 );
} );
Copy link
Author

Choose a reason for hiding this comment

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

I find this test extremely suspect, and I doubt it

I have trouble understanding your structure because I'm not clear on what a "reflection" is

Copy link
Member

@GerkinDev GerkinDev Jun 16, 2022

Choose a reason for hiding this comment

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

A refkection is none of my concept : it's directly inherited from typedoc structure, which is a pain in the a**.
Basically any objects in your project, frol the project itself, to const, symboks, type defs, & anything that is output to the docs is a reflection.

return node.children ?
this._mapPagesToReflections( node.children, parent, childrenIO ) :
[];
}
Copy link
Author

Choose a reason for hiding this comment

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

I'm genuinely not clear on whether this actually works. Once building works again, I'll go kick it until it plays nice

@StoneCypher
Copy link
Author

I accidentally presented more in this PR than I had meant to; this should be two PRs. Sorry; I botched remaking them after --no-verify

Sending anew, momentarily

@StoneCypher StoneCypher deleted the FixScripts branch June 15, 2022 18:16
@StoneCypher
Copy link
Author

StoneCypher commented Jun 15, 2022

This is what I had meant to send. #88

@StoneCypher
Copy link
Author

This is the extra part that I accidentally stacked in. It needs to be distinct because that part isn't yet ready. #89

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.

2 participants