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

we don't show timing in the main log output #368

Open
yegor256 opened this issue Nov 4, 2024 · 2 comments · May be fixed by #389
Open

we don't show timing in the main log output #368

yegor256 opened this issue Nov 4, 2024 · 2 comments · May be fixed by #389
Labels
bug Something isn't working enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@yegor256
Copy link
Member

yegor256 commented Nov 4, 2024

Currently, I'm getting this:

$ eoc compile
EO objects registered in .eoc/eo-foreign.json
EO program assembled in .eoc
EO program verified in .eoc
Java sources generated in .eoc/generated-sources
Java .class files compiled into .eoc

Would be much better to see this:

EO objects registered in .eoc/eo-foreign.json in 34ms
EO program assembled in .eoc in 1s
EO program verified in .eoc in 340ms
Java sources generated in .eoc/generated-sources in 5s
Java .class files compiled into .eoc in 2s

Let's add the timings at the end of every logging line.
@yegor256 yegor256 added enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed labels Nov 4, 2024
@yegor256 yegor256 changed the title show timing in the main log output we don't show timing in the main log output Nov 4, 2024
@yegor256 yegor256 added the bug Something isn't working label Nov 4, 2024
@ReBy298
Copy link

ReBy298 commented Nov 4, 2024

I would like to contribute to this issue :D

@yegor256
Copy link
Member Author

yegor256 commented Nov 5, 2024

@ReBy298 please, go head!

@ReBy298 ReBy298 mentioned this issue Nov 7, 2024
Closed
@trapvpack trapvpack mentioned this issue Nov 23, 2024
Open
trapvpack added a commit to trapvpack/eoc that referenced this issue Nov 23, 2024
@trapvpack
fix(objectionary#368): added utility function for elapsed time tracking
0fac2c1
@trapvpack trapvpack linked a pull request Nov 23, 2024 that will close this issue
Open
trapvpack added a commit to trapvpack/eoc that referenced this issue Nov 23, 2024
@trapvpack
fix(objectionary#368): simplify logic and format code for elapsed
920198b
trapvpack added a commit to trapvpack/eoc that referenced this issue Nov 23, 2024
@trapvpack
fix(objectionary#368): fix up todo needful time
32e0759
trapvpack added a commit to trapvpack/eoc that referenced this issue Nov 24, 2024
@trapvpack
fix(objectionary#368): formatted, simplified
80ee2ec
trapvpack added a commit to trapvpack/eoc that referenced this issue Nov 25, 2024
@trapvpack
fix(objectionary#368): formatted
4efeacb
trapvpack added a commit to trapvpack/eoc that referenced this issue Nov 26, 2024
@trapvpack
fix(objectionary#368): added utility function for elapsed time tracking
0911954
trapvpack added a commit to trapvpack/eoc that referenced this issue Nov 26, 2024
@trapvpack
fix(objectionary#368): simplify logic and format code for elapsed
62c60f3
trapvpack added a commit to trapvpack/eoc that referenced this issue Nov 26, 2024
@trapvpack
fix(objectionary#368): fix up todo needful time
42858f3
trapvpack added a commit to trapvpack/eoc that referenced this issue Nov 26, 2024
@trapvpack
fix(objectionary#368): formatted, simplified
357691e
trapvpack added a commit to trapvpack/eoc that referenced this issue Nov 26, 2024
@trapvpack
fix(objectionary#368): formatted
4abfb9c
trapvpack added a commit to trapvpack/eoc that referenced this issue Nov 26, 2024
@trapvpack
fix(objectionary#368): formatted
5834769
trapvpack added a commit to trapvpack/eoc that referenced this issue Nov 26, 2024
@trapvpack
fix(objectionary#368): formatted
816cb0f
trapvpack added a commit to trapvpack/eoc that referenced this issue Nov 26, 2024
@trapvpack
fix(objectionary#368): formatted
b37f5bf
trapvpack added a commit to trapvpack/eoc that referenced this issue Nov 26, 2024
@trapvpack
fix(objectionary#368): formatted
7ef9618
trapvpack added a commit to trapvpack/eoc that referenced this issue Nov 27, 2024
@trapvpack
fix(objectionary#368): formatted
4dd5eea
trapvpack added a commit to trapvpack/eoc that referenced this issue Nov 27, 2024
@trapvpack
fix(objectionary#368): formatted
9d3e2a8
trapvpack added a commit to trapvpack/eoc that referenced this issue Nov 27, 2024
@trapvpack
fix(objectionary#368): simplify regex for time measurement in test
f911833
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(#368): added utility function for elapsed time tracking trapvpack/eoc
2 participants
@yegor256 @ReBy298