Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix broken link to most recent jest config example #9266

Merged
merged 2 commits into from
Oct 6, 2023

Conversation

Philzen
Copy link
Contributor

@Philzen Philzen commented Oct 4, 2023

No description provided.

@Tobbe Tobbe added the release:docs This PR only updates docs label Oct 4, 2023
@Tobbe Tobbe added this to the next-release-patch milestone Oct 4, 2023
@Tobbe Tobbe enabled auto-merge (squash) October 4, 2023 20:11
@Philzen
Copy link
Contributor Author

Philzen commented Oct 4, 2023

@Tobbe Just stumbled over this b/c actually i was trying to find the root cause for this kind of noisy debug output appearing in my api-side tests all the time

{"level":30,"time":1696450323107,"pid":177559,"hostname":"machine-Philzen","prisma":{"clientVersion":"4.12.0"},"timestamp":"2023-10-04T20:12:03.106Z","message":"Starting a sqlite pool with 5 connections.","target":"quaint::pooled","msg":"Starting a sqlite pool with 5 connections."}

I understand now this comes from the trace debug level – which would be tolerable but it seems that when running jest it seems (at least for me) the rw-log-formatter is not active. Also https://redwoodjs.com/docs/logger does not have any mention of jest.

Is there an open issue i can hook into or is this a new one? Sorry for abusing this PR for the off-topic, well, X-Y-problem topic crossposting.

Copy link
Member

Tobbe commented Oct 5, 2023

@Philzen l don't know of an existing issue. Please do create a new one.

@jtoar jtoar modified the milestones: next-release-patch, v6.3.2 Oct 5, 2023
@jtoar jtoar disabled auto-merge October 6, 2023 18:38
@jtoar jtoar merged commit a8f70ee into redwoodjs:main Oct 6, 2023
@Philzen Philzen deleted the patch-17 branch December 23, 2024 01:08
@Philzen
Copy link
Contributor Author

Philzen commented Dec 23, 2024

@Philzen l don't know of an existing issue. Please do create a new one.

#11757

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:docs This PR only updates docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants