Skip to content

Conversation

hugomrdias
Copy link
Member

@hugomrdias hugomrdias commented Jan 25, 2021

Changes:

88a6a0c (Hugo Dias, 56 seconds ago)
fix: move ts docs to docs cmd and improve config

  • move ts -p docs cmd to docs cmd directly
  • start proper config merging between config file and cmd flags
    - updated cmds: build, docs, lint and ts, more will be updated in the future
  • add new options for docs entryPoint
  • user config uses lilconfig to parse and find the local file or
    package.json

f629a3c (Hugo Dias, 9 minutes ago)
fix: update to latest typedoc

  • fix some internal calls to ts cmd
  • new config and plugin for typedoc 0.20

BREAKING CHANGE: aegir ts -p docs removed use aegir docs directly

- fix some internal calls to ts cmd
- new config and plugin for typedoc 0.20
- move ts -p doc cmd to docs cmd directly
- start proper config merging between config file and cmd flags
  - build, docs, lint and ts cmds, more will be updated in the future
- add new options for docs `entryPoint`
- user config uses `lilconfig` to parse and find the local file or package.json
Copy link
Member

@vasco-santos vasco-santos left a comment

Choose a reason for hiding this comment

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

Looking good, left a couple of questions before approval

type: 'boolean',
describe: 'Enable support for Typescript',
default: false
default: userConfig.ts // needs to be changed to another name to not conflict with the ts cmd options
Copy link
Member

Choose a reason for hiding this comment

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

Is this needed now?

Copy link
Member Author

Choose a reason for hiding this comment

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

yes this enables support for TS repos, actual TS code not like ours which are JS with JSDocs types

@hugomrdias hugomrdias merged commit 1149281 into master Jan 26, 2021
@hugomrdias hugomrdias deleted the feat/docs2.0 branch January 26, 2021 19:12
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