Skip to content

Move --printUnusedTemplates in hugo scripts#38227

Merged
XhmikosR merged 1 commit intomainfrom
xmr/printUnusedTemplates
Mar 14, 2023
Merged

Move --printUnusedTemplates in hugo scripts#38227
XhmikosR merged 1 commit intomainfrom
xmr/printUnusedTemplates

Conversation

@XhmikosR
Copy link
Copy Markdown
Member

Description

Motivation & Context

Type of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactoring (non-breaking change)
  • Breaking change (fix or feature that would change existing functionality)

Checklist

  • I have read the contributing guidelines
  • My code follows the code style of the project (using npm run lint)
  • My change introduces changes to the documentation
  • I have updated the documentation accordingly
  • I have added tests to cover my changes
  • All new and existing tests passed

Live previews

Related issues


@julien-deramond LGTY? Generally, we should try to not add new scripts if we can get away with the existent ones, because the list is already too long :)

@XhmikosR XhmikosR force-pushed the xmr/printUnusedTemplates branch from 276931a to f92770a Compare March 14, 2023 06:52
@julien-deramond julien-deramond self-requested a review March 14, 2023 06:55
Copy link
Copy Markdown
Member

@julien-deramond julien-deramond left a comment

Choose a reason for hiding this comment

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

LGTM :). As long as we can have the info with the "serve" and/or "build" commands, I'm good with it.

@XhmikosR XhmikosR marked this pull request as ready for review March 14, 2023 06:57
@XhmikosR XhmikosR merged commit ca6dfad into main Mar 14, 2023
@XhmikosR XhmikosR deleted the xmr/printUnusedTemplates branch March 14, 2023 07:09
@mahilanmjd mahilanmjd mentioned this pull request Apr 16, 2023
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants