Skip to content

Update README.md#5

Open
dphdave wants to merge 4 commits into
SketchUp:mainfrom
dphdave:patch-1
Open

Update README.md#5
dphdave wants to merge 4 commits into
SketchUp:mainfrom
dphdave:patch-1

Conversation

@dphdave

@dphdave dphdave commented Nov 10, 2022

Copy link
Copy Markdown

Make setup instructions a little more user-friendly!

Make setup instructions a little more user-friendly!
Add a warning when trying to load code that was pasted from the Ruby Console (hence has no source location)
Comment thread README.md
![Profiling in SketchUp](docs/images/profiling.png)

## Installation
There is no RBZ file! Download the ZIP archive of the code from https://github.com/SketchUp/speedup/archive/refs/heads/main.zip

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would encourage to clone the repo instead of downloading it.

Though, we should start adding RBZ files for each release.

path = self.instance_method(random_profile_method).source_location.first
$VERBOSE = nil
load path
if File.exist?(path)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When would you run into this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants