Skip to content

Conversation

timholy
Copy link
Member

@timholy timholy commented Apr 5, 2019

The signature is the same, but the "file" may be different. Consequently we shouldn't do signature-based lookup, we should simply look it up again.

A test is already prepared for Revise, will be submitted after this gets tagged.

The signature is the same, but the "file" may be different.
Consequently we shouldn't do signature-based lookup, we should simply
look it up again.
@timholy
Copy link
Member Author

timholy commented Apr 5, 2019

Oops, meant fixes #38

@codecov
Copy link

codecov bot commented Apr 5, 2019

Codecov Report

Merging #39 into master will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@         Coverage Diff          @@
##           master   #39   +/-   ##
====================================
  Coverage       0%    0%           
====================================
  Files           3     3           
  Lines         163   165    +2     
====================================
- Misses        163   165    +2
Impacted Files Coverage Δ
src/CodeTracking.jl 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 a408c96...88f5ce2. Read the comment docs.

@timholy timholy merged commit 8092951 into master Apr 5, 2019
@timholy timholy deleted the teh/fix_38 branch April 5, 2019 17:55
timholy added a commit to timholy/Revise.jl that referenced this pull request Apr 5, 2019
timholy added a commit to timholy/Revise.jl that referenced this pull request Apr 5, 2019
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.

1 participant