Skip to content
This repository has been archived by the owner on Jan 11, 2024. It is now read-only.

Use os.signpost and os.log for better logging and profiling #50

Merged
merged 3 commits into from Oct 29, 2018

Conversation

toshi0383
Copy link
Owner

Now we can visualize network requests in instruments' "os_signpost" trace.
Network request/response performance is automatically logged via os_signpost.
screen shot 2018-10-26 at 11 24 14

@codecov-io
Copy link

codecov-io commented Oct 26, 2018

Codecov Report

Merging #50 into master will increase coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #50      +/-   ##
=========================================
+ Coverage    4.96%   4.96%   +<.01%     
=========================================
  Files          40      40              
  Lines        1692    1691       -1     
=========================================
  Hits           84      84              
+ Misses       1608    1607       -1
Impacted Files Coverage Δ
...Sources/List/Builds/BuildsListViewController.swift 0% <ø> (ø) ⬆️
...ent/Sources/List/Builds/BuildsPollingManager.swift 0% <0%> (ø) ⬆️
...ient/Sources/List/Builds/BuildsListViewModel.swift 0% <0%> (ø) ⬆️
BitriseClient/Sources/Extension/APIKit+Rx.swift 0% <0%> (ø) ⬆️
...lient/Sources/BitriseYml/BitriseYmlViewModel.swift 0% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f42048a...b884b5b. Read the comment docs.

@toshi0383 toshi0383 changed the title Use os.signpost for better logging and profiling Use os.signpost and os.log for better logging and profiling Oct 27, 2018
@toshi0383 toshi0383 merged commit 32e1521 into master Oct 29, 2018
@toshi0383 toshi0383 deleted the signpost branch October 29, 2018 00:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants