Skip to content

fix(scully): fix defaultPostRenderers type#559

Merged
SanderElias merged 3 commits intoscullyio:masterfrom
puku0x:fix-scully-config-type
May 18, 2020
Merged

fix(scully): fix defaultPostRenderers type#559
SanderElias merged 3 commits intoscullyio:masterfrom
puku0x:fix-scully-config-type

Conversation

@puku0x
Copy link
Copy Markdown
Contributor

@puku0x puku0x commented May 17, 2020

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Other... Please describe:

What is the current behavior?

Linter says "defaultPostRenderers is missing" which is an optional array.
https://github.com/scullyio/scully/blob/master/libs/scully/src/lib/utils/config.ts#L74

image

What is the new behavior?

There should be no errors after the config file is generated.

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

The document for the interface is synchronized to the latest ScullyConfig.

@SanderElias SanderElias merged commit 79d4c1e into scullyio:master May 18, 2020
@puku0x
Copy link
Copy Markdown
Contributor Author

puku0x commented May 18, 2020

@SanderElias Thank you!

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