Skip to content

fix(gatsby-plugin-sitemap): Remove reporter.verbose calls#31448

Merged
LekoArts merged 1 commit intomasterfrom
sitemap-verbose-remove
May 17, 2021
Merged

fix(gatsby-plugin-sitemap): Remove reporter.verbose calls#31448
LekoArts merged 1 commit intomasterfrom
sitemap-verbose-remove

Conversation

@LekoArts
Copy link
Copy Markdown
Contributor

Description

Gatsby Cloud is using the verbose mode and on a large site (even with 500 pages) logging everything makes the "Raw logs" unusable

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label May 17, 2021
@LekoArts LekoArts removed the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label May 17, 2021
Copy link
Copy Markdown
Contributor

@pieh pieh left a comment

Choose a reason for hiding this comment

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

We really need more flexibility with the debug/verbose messages, single true/false is not enough :(

In the meantime, LGTM! Thanks!

@LekoArts LekoArts added the bot: merge on green Gatsbot will merge these PRs automatically when all tests passes label May 17, 2021
@LekoArts LekoArts merged commit c339f67 into master May 17, 2021
@LekoArts LekoArts deleted the sitemap-verbose-remove branch May 17, 2021 12:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bot: merge on green Gatsbot will merge these PRs automatically when all tests passes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants