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

Move process.log into a single file in the root of the artifacts folder #1445

Open
Tracked by #4233
LeoNatan opened this issue Jun 12, 2019 · 7 comments
Open
Tracked by #4233

Comments

@LeoNatan
Copy link
Contributor

LeoNatan commented Jun 12, 2019

As discussed offline, the logic should just output all process logs into a single process.log, rather than cutting it into chunks (with potential to mess up).

The predicate should be changed into process==$processName, where processName is taken before start from the CFBundleExecutable Info.plist of the .app specified in package.json.

@LeoNatan
Copy link
Contributor Author

The process==${} has been implemented already in #1877.

@stale
Copy link

stale bot commented May 25, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
If you believe the issue is still relevant, please test on the latest Detox and report back.

Thank you for your contributions!

For more information on bots in this reporsitory, read this discussion.

@stale stale bot added the 🏚 stale label May 25, 2022
@d4vidi d4vidi closed this as completed May 26, 2022
@d4vidi d4vidi reopened this May 26, 2022
@d4vidi d4vidi closed this as not planned Won't fix, can't repro, duplicate, stale May 26, 2022
@stale stale bot removed the 🏚 stale label May 26, 2022
@noomorph
Copy link
Collaborator

noomorph commented May 29, 2022

@d4vidi, planned in V20 (detox_pid_NNNN.log merge). Or this is about device.log??

@noomorph noomorph reopened this May 29, 2022
@d4vidi
Copy link
Collaborator

d4vidi commented Jun 6, 2022

This was about device.log (previously named process.log...). I don't think there's a case for this anymore.

@stale
Copy link

stale bot commented Jul 8, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
If you believe the issue is still relevant, please test on the latest Detox and report back.

Thank you for your contributions!

For more information on bots in this reporsitory, read this discussion.

@stale stale bot added the 🏚 stale label Jul 8, 2022
@stale
Copy link

stale bot commented Jul 25, 2022

The issue has been closed for inactivity.

@stale stale bot closed this as completed Jul 25, 2022
@noomorph
Copy link
Collaborator

Regarding the complete device (app) log – looks like it can be done in the scope of the next Artifacts refactor, in late stages of Remote Detox - #3743.

Related to #3092.

@noomorph noomorph reopened this Jan 11, 2023
@stale stale bot removed 🏚 stale labels Jan 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants