Skip to content

DEV: Flip --raw-logs to default true and add --timestamps option.#13944

Merged
Grubba27 merged 6 commits intodevelfrom
dev/flip-raw-logs-variable
Oct 13, 2025
Merged

DEV: Flip --raw-logs to default true and add --timestamps option.#13944
Grubba27 merged 6 commits intodevelfrom
dev/flip-raw-logs-variable

Conversation

@Grubba27
Copy link
Copy Markdown
Contributor

@Grubba27 Grubba27 commented Sep 30, 2025

Screenshot 2025-09-29 at 22 30 40

This for sure will make the regular dev xp better; this could arguably be a breaking change but I really want to believe that people did not rely on this timestamp for their logs (I assume people are using the logging package)

Screenshot 2025-09-29 at 22 34 33 Screenshot 2025-09-29 at 22 35 12

relates to: https://forums.meteor.com/t/meteor-development-tiny-tip-that-helped-me-a-lot/64156/4?u=grubba

@netlify
Copy link
Copy Markdown

netlify bot commented Sep 30, 2025

Deploy Preview for v3-meteor-api-docs ready!

Name Link
🔨 Latest commit 6541c65
🔍 Latest deploy log https://app.netlify.com/projects/v3-meteor-api-docs/deploys/68ecf8a828490f00083d8a7e
😎 Deploy Preview https://deploy-preview-13944.docs-online.meteor.com
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link
Copy Markdown

netlify bot commented Sep 30, 2025

Deploy Preview for v3-migration-docs canceled.

Name Link
🔨 Latest commit 6541c65
🔍 Latest deploy log https://app.netlify.com/projects/v3-migration-docs/deploys/68ecf8a8e12d790008b15bfe

@harryadel
Copy link
Copy Markdown
Contributor

Thank you @Grubba27 🎉

@StorytellerCZ StorytellerCZ added this to the Release 3.4 milestone Sep 30, 2025
@StorytellerCZ
Copy link
Copy Markdown
Collaborator

Yes for this and the logging package will need improvement in general. I would like to imagine it in the future that it could connect to outside logging services as well.

@zodern
Copy link
Copy Markdown
Collaborator

zodern commented Sep 30, 2025

One benefit from the timestamp is it was obvious what logs were from the meteor-tool or build plugins, and what logs were from the app. For example, in the screenshot of the error, it's not obvious if that error happened while building the app, or when running the app. Would there be any value in keeping some sort of indicator/prefix for app logs (or maybe instead having a prefix for the meteor-tool/build-plugin logs)?

I assume people are using the logging package

Maybe other developers have had a different experience, but I've never personally seen an app that directly used the logging package.

@harryadel
Copy link
Copy Markdown
Contributor

Maybe other developers have had a different experience, but I've never personally seen an app that directly used the logging package.

Me neither.

@Grubba27
Copy link
Copy Markdown
Contributor Author

For galaxy users, the logging package makes their logs look good:

check this forum thread: https://forums.meteor.com/t/suggestions-for-logging-in-galaxy/63119

@Grubba27
Copy link
Copy Markdown
Contributor Author

Would there be any value in keeping some sort of indicator/prefix for app logs (or maybe instead having a prefix for the meteor-tool/build-plugin logs)?

I think having a better-defined prefix for the meteor-tool would be better. Currently, we use =>, but I think if we added some color, emoji (maybe 🚀 / ☄️ ) or something to define the tool would be better. Also, our tool should behave a bit more like Vite and others (in the sense of not clogging the logs but actually replacing text)

cc @nachocodoner

@italojs
Copy link
Copy Markdown
Member

italojs commented Sep 30, 2025

Would there be any value in keeping some sort of indicator/prefix for app logs (or maybe instead having a prefix for the meteor-tool/build-plugin logs)?

A prefix would have the same issue as timestamp: use case

We can set it as optional or improve what we have today as you mentioned, color, emoji..., but I don't see any value on adding prefix docker-compose like here since that log is more about Meteor's stuff, am I right? I just see a prefix being used by the application, but the dev can do it

@StorytellerCZ
Copy link
Copy Markdown
Collaborator

For a very long time the logging package was not documented until I took interest in it and wrote the documentation. This is why not so many people use it. I think it deserves much more love (as in usage and development).

https://docs.meteor.com/packages/logging.html

@Grubba27
Copy link
Copy Markdown
Contributor Author

Grubba27 commented Oct 1, 2025

Windows checks are failing 😭

@nachocodoner
Copy link
Copy Markdown
Member

Great to see this change moving forward. Not sure why Windows is failing. We need to investigate, it seems related to environment setup, not to these changes.

@Grubba27 Grubba27 merged commit 6f48b86 into devel Oct 13, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants