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.

Sign up for GitHub

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

Jump to bottom

Feature request: logMessageSuffix #128

Closed
mariaedwards opened this issue Apr 9, 2021 · 14 comments
Closed

Feature request: logMessageSuffix #128

mariaedwards opened this issue Apr 9, 2021 · 14 comments

Comments

@mariaedwards
Copy link

Could you please add a new property logMessageSuffix and default it to :?
So when console.log is added it will be

🟣 ~ file: app.js ~ line 6 ~ second: Second file

instead of:

🟣 ~ file: app.js ~ line 6 ~ second Second file
@tfreitas90
Copy link

This would be nice....

@besnikkorca besnikkorca mentioned this issue Sep 27, 2022
Open
@besnikkorca
Copy link

Added PR that should allow this @mariaedwards @tfreitas90

@farhadh
Copy link
Contributor

farhadh commented Jan 21, 2023

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 :)

@Chakroun-Anas
Copy link
Owner

Chakroun-Anas commented Jan 22, 2023 via email

@farhadh
Copy link
Contributor

farhadh commented Jan 22, 2023

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...

@Chakroun-Anas
Copy link
Owner

Chakroun-Anas commented Jan 28, 2023 via email

@farhadh
Copy link
Contributor

farhadh commented Jan 28, 2023

Hi @Chakroun-Anas!

Yes. Just let me know when :)

@Chakroun-Anas
Copy link
Owner

Chakroun-Anas commented Jan 30, 2023 via email

@farhadh
Copy link
Contributor

farhadh commented Jan 31, 2023

It's ok, @Chakroun-Anas. Why though? :)) You can see it in my profile ;)

@Chakroun-Anas
Copy link
Owner

Chakroun-Anas commented Feb 1, 2023 via email

@farhadh
Copy link
Contributor

farhadh commented Feb 3, 2023

@Chakroun-Anas OK. Would you mind creating the Slack channel and letting me know about it? :)

@Chakroun-Anas
Copy link
Owner

Chakroun-Anas commented Feb 3, 2023 via email

@farhadh
Copy link
Contributor

farhadh commented Feb 5, 2023

@Chakroun-Anas I sent you and email :)

@Chakroun-Anas
Copy link
Owner

Hello guys,

Done by @farhadh , thank you so much for your efforts ^^

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@farhadh @besnikkorca @Chakroun-Anas @mariaedwards @tfreitas90