Closed mariaedwards closed 1 year ago
This would be nice....
Added PR that should allow this @mariaedwards @tfreitas90
Hey!
I have added the PR for this feature for the most recent refactored code :)
I hope @Chakroun-Anas would confirm and merge it <3. I too would like to have this feature so badly :)
Hello Farhad,
Thank you for your contribution, did check that all the tests pass ? ^^
Cordially,
De : Farhad H. P. Shirvan @.> EnvoyΓ© : samedi 21 janvier 2023 15:21 Γ : Chakroun-Anas/turbo-console-log @.> Cc : Chakroun Anas @.>; Mention @.> Objet : Re: [Chakroun-Anas/turbo-console-log] Feature request: logMessageSuffix (#128)
Hey!
I have added the PRhttps://github.com/Chakroun-Anas/turbo-console-log/pull/190 for this feature for the most recent refactored code :)
I hope @Chakroun-Anashttps://github.com/Chakroun-Anas would confirm and merge it <3. I too would like to have this feature so badly :)
β Reply to this email directly, view it on GitHubhttps://github.com/Chakroun-Anas/turbo-console-log/issues/128#issuecomment-1399260731, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIRYHC2CS6OWJBYUSDBWD3TWTPWIHANCNFSM42V3BNTQ. You are receiving this because you were mentioned.Message ID: @.***>
Hi @Chakroun-Anas!
My pleasure. Yes. All tests, as shown below, were passed for my 3 PRs (#190, #191, and #192) which are on top of each other :)
Completed initializing default profile extensions in extensions installation folder. file:///home/farhad/.vscode/extensions
Loading development extension at /home/farhad/turbo/turbo-console-log
β Insert log message related to a class constructor param (3636ms)
β Insert log message related to a class function param (738ms)
β Insert log message related to a function parameter within a class function (654ms)
β Insert log message related to a deconstructred property assigned to a variable (977ms)
β Insert log message related to a multiline function parameter (2072ms)
β Insert log message related to a function parameter (546ms)
[main 2023-01-22T16:55:31.807Z] update#setState checking for updates
[main 2023-01-22T16:55:31.908Z] update#setState idle
β Insert log message in the context of a function with a bunch of decorators (515ms)
β Determine the closing brace line num (286ms)
β Insert log message in the last line of the file (451ms)
β Insert log message related to an object variable (350ms)
β Insert log message related to an object function call (532ms)
β Obj function call no variable assignment (287ms)
β Object variable with a declared type (545ms)
β Insert log message related to a function (which params are defined in one line) param (312ms)
β Insert log message related to a primitive variable (402ms)
β Insert log message related to a variable declared in a promise callback (477ms)
d
Single line
β Assigned to a variable
β Parameter of a function
β Chained call
Multiple lines
β Example 01
β Example 02
β Example 03
β Example 04
Empty functions
β Example 01
β Example 02
Insert log message related to a variable with a function as the value
β Example 01
β Example 02
β Example 03
JS Line Code Processing
β Assignment of object literal to a variable
β Assignment of an array to a variable
Classes
β Check class declaration
β Get class name
Functions
β Check built in function invocation
β Check named function Declaration
β Get function name
β Assignment of function to a variable
β Check object function call
β Anonymous functions
β Arugment of anonymous function
β Should transform anonymous function
[main 2023-01-22T16:55:56.039Z] Extension host with pid 7170 exited with code: 0, signal: null.
Exit code: 0
Done
Waiting for the debugger to disconnect...
Waiting for the debugger to disconnect...
Hello Farhad,
Do you have some time to talk this weekend ?
Thx
De : Anas Chakroun @.> EnvoyΓ© : dimanche 22 janvier 2023 16:52 Γ : Chakroun-Anas/turbo-console-log @.> Objet : RE: [Chakroun-Anas/turbo-console-log] Feature request: logMessageSuffix (#128)
Hello Farhad,
Thank you for your contribution, did check that all the tests pass ? ^^
Cordially,
De : Farhad H. P. Shirvan @.> EnvoyΓ© : samedi 21 janvier 2023 15:21 Γ : Chakroun-Anas/turbo-console-log @.> Cc : Chakroun Anas @.>; Mention @.> Objet : Re: [Chakroun-Anas/turbo-console-log] Feature request: logMessageSuffix (#128)
Hey!
I have added the PRhttps://github.com/Chakroun-Anas/turbo-console-log/pull/190 for this feature for the most recent refactored code :)
I hope @Chakroun-Anashttps://github.com/Chakroun-Anas would confirm and merge it <3. I too would like to have this feature so badly :)
β Reply to this email directly, view it on GitHubhttps://github.com/Chakroun-Anas/turbo-console-log/issues/128#issuecomment-1399260731, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIRYHC2CS6OWJBYUSDBWD3TWTPWIHANCNFSM42V3BNTQ. You are receiving this because you were mentioned.Message ID: @.***>
Hi @Chakroun-Anas!
Yes. Just let me know when :)
Hello,
So sorry, i barely find some time, where are you from i may to ask?
Cordially,
De : Farhad H. P. Shirvan @.> EnvoyΓ© : samedi 28 janvier 2023 11:58 Γ : Chakroun-Anas/turbo-console-log @.> Cc : Chakroun Anas @.>; Mention @.> Objet : Re: [Chakroun-Anas/turbo-console-log] Feature request: logMessageSuffix (#128)
Hi @Chakroun-Anashttps://github.com/Chakroun-Anas!
Yes. Just let me know when :)
β Reply to this email directly, view it on GitHubhttps://github.com/Chakroun-Anas/turbo-console-log/issues/128#issuecomment-1407373532, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIRYHCZMBZPPFIFBEKPA7Z3WUT3TVANCNFSM42V3BNTQ. You are receiving this because you were mentioned.Message ID: @.***>
It's ok, @Chakroun-Anas. Why though? :)) You can see it in my profile ;)
Hello,
What do you think about creating a slack channel for the extension so that we can communicate more efficiently, i need to talk to you before getting to collaborate please ^^
Thx
De : Farhad H. P. Shirvan @.> EnvoyΓ© : mardi 31 janvier 2023 12:10 Γ : Chakroun-Anas/turbo-console-log @.> Cc : Chakroun Anas @.>; Mention @.> Objet : Re: [Chakroun-Anas/turbo-console-log] Feature request: logMessageSuffix (#128)
It's ok, @Chakroun-Anashttps://github.com/Chakroun-Anas. Why though? :)) You can see it in my profile ;)
β Reply to this email directly, view it on GitHubhttps://github.com/Chakroun-Anas/turbo-console-log/issues/128#issuecomment-1410169199, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIRYHC5VKI7GXRGNSQLQXN3WVDXLNANCNFSM42V3BNTQ. You are receiving this because you were mentioned.Message ID: @.***>
@Chakroun-Anas OK. Would you mind creating the Slack channel and letting me know about it? :)
Hello,
I created the channel π
I need your email so that i can invite you and get back to work π
Thx,
De : Farhad H. P. Shirvan @.> EnvoyΓ© : vendredi 3 fΓ©vrier 2023 11:34 Γ : Chakroun-Anas/turbo-console-log @.> Cc : Chakroun Anas @.>; Mention @.> Objet : Re: [Chakroun-Anas/turbo-console-log] Feature request: logMessageSuffix (#128)
@Chakroun-Anashttps://github.com/Chakroun-Anas OK. Would you mind creating the Slack channel and letting me know about it? :)
β Reply to this email directly, view it on GitHubhttps://github.com/Chakroun-Anas/turbo-console-log/issues/128#issuecomment-1415656961, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIRYHC4QJ4IW2BLGXLCBEUTWVTNMLANCNFSM42V3BNTQ. You are receiving this because you were mentioned.Message ID: @.***>
@Chakroun-Anas I sent you and email :)
Hello guys,
Done by @farhadh , thank you so much for your efforts ^^
Could you please add a new property logMessageSuffix and default it to
:
? So when console.log is added it will beinstead of: