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

feat:add timestamp to log printing #308

Merged
merged 1 commit into from
Oct 25, 2023
Merged

feat:add timestamp to log printing #308

merged 1 commit into from
Oct 25, 2023

Conversation

DanLi39
Copy link
Collaborator

@DanLi39 DanLi39 commented Oct 24, 2023

Fix #234

@DanLi39 DanLi39 added the enhancement ⭐ New feature or request label Oct 24, 2023
@codecov
Copy link

codecov bot commented Oct 24, 2023

Codecov Report

Merging #308 (0e03759) into main (18fcfc6) will increase coverage by 3.47%.
Report is 62 commits behind head on main.
The diff coverage is 68.57%.

@@             Coverage Diff              @@
##               main     #308      +/-   ##
============================================
+ Coverage     56.17%   59.64%   +3.47%     
- Complexity     1899     2323     +424     
============================================
  Files           263      312      +49     
  Lines         10097    11707    +1610     
  Branches       1148     1370     +222     
============================================
+ Hits           5672     6983    +1311     
- Misses         3913     4084     +171     
- Partials        512      640     +128     
Files Coverage Δ
...java/io/arex/agent/bootstrap/AgentInitializer.java 85.18% <100.00%> (+1.18%) ⬆️
.../arex/agent/bootstrap/cache/LoadedModuleCache.java 71.42% <100.00%> (ø)
...o/arex/agent/bootstrap/model/MockStrategyEnum.java 0.00% <ø> (ø)
...ain/java/io/arex/agent/bootstrap/util/IOUtils.java 100.00% <100.00%> (ø)
.../java/io/arex/agent/bootstrap/util/StringUtil.java 65.87% <100.00%> (+1.71%) ⬆️
...gent/instrumentation/InstrumentationInstaller.java 80.20% <100.00%> (-0.98%) ⬇️
...java/io/arex/inst/extension/ModuleDescription.java 95.23% <100.00%> (+23.80%) ⬆️
...x/inst/extension/matcher/ModuleVersionMatcher.java 92.30% <100.00%> (ø)
.../main/java/io/arex/inst/runtime/config/Config.java 90.41% <100.00%> (+0.70%) ⬆️
...java/io/arex/inst/runtime/context/ArexContext.java 31.03% <100.00%> (+5.57%) ⬆️
... and 113 more

... and 10 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@DanLi39 DanLi39 force-pushed the feature/addTimeStamp branch 11 times, most recently from bb90bb0 to 8805cc5 Compare October 25, 2023 12:04
@DanLi39 DanLi39 force-pushed the feature/addTimeStamp branch from 8805cc5 to 0e03759 Compare October 25, 2023 12:05
@sonarqubecloud
Copy link

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell B 3 Code Smells

85.7% 85.7% Coverage
0.0% 0.0% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

@DanLi39 DanLi39 merged commit d26b842 into main Oct 25, 2023
7 of 8 checks passed
@DanLi39 DanLi39 deleted the feature/addTimeStamp branch October 25, 2023 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement ⭐ New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

这些日志打印能否默认带上时间戳呢?
3 participants