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

Small Remotery tutorials fixes #92

Merged
merged 1 commit into from
Sep 4, 2020
Merged

Conversation

JShep1
Copy link

@JShep1 JShep1 commented Sep 4, 2020

Just some small fixes that I came across while going through this tutorial. I also added a line more explicitly indicated to run ign_remotery_vis for Ignition libraries as it wasn't immediately clear to me how to open the browser window when profiling Ignition libraries

Signed-off-by: John Shepherd [email protected]

Signed-off-by: John Shepherd <[email protected]>
@github-actions github-actions bot added 🏰 citadel Ignition Citadel 📜 blueprint Ignition Blueprint 🔮 dome Ignition Dome labels Sep 4, 2020
@codecov
Copy link

codecov bot commented Sep 4, 2020

Codecov Report

Merging #92 into ign-common3 will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##           ign-common3      #92   +/-   ##
============================================
  Coverage        73.94%   73.94%           
============================================
  Files               69       69           
  Lines             9390     9390           
============================================
  Hits              6943     6943           
  Misses            2447     2447           
Impacted Files Coverage Δ
profiler/src/RemoteryProfilerImpl.cc 19.31% <100.00%> (ø)

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 c03ede0...2bd9247. Read the comment docs.

@chapulina chapulina merged commit fcf1d2c into ign-common3 Sep 4, 2020
@chapulina chapulina deleted the jshep1/small_tutorial_fix branch September 4, 2020 20:35
mjcarroll pushed a commit that referenced this pull request Oct 14, 2020
Signed-off-by: John Shepherd <[email protected]>
mjcarroll pushed a commit that referenced this pull request Oct 19, 2020
Signed-off-by: John Shepherd <[email protected]>
Signed-off-by: Michael Carroll <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📜 blueprint Ignition Blueprint 🏰 citadel Ignition Citadel 🔮 dome Ignition Dome
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants