-
Notifications
You must be signed in to change notification settings - Fork 115
Comparing changes
Open a pull request
base repository: timholy/Revise.jl
base: v3.7.6
head repository: timholy/Revise.jl
compare: v3.8.0
- 12 commits
- 18 files changed
- 2 contributors
Commits on May 19, 2025
-
remove no-longer necessary
moduledepsdata structure (#916)This data structure was added in 32f8727, but its functionality has since been provided by LoweredCodeUtils and is no longer needed. This removes all code related to `moduledeps` and clean up.
Configuration menu - View commit details
-
Copy full SHA for 4a0570d - Browse repository at this point
Copy the full SHA 4a0570dView commit details -
some minor refactoring on
methods_by_execution!(#917)Separate functions based on what they return.
Configuration menu - View commit details
-
Copy full SHA for d9aecc2 - Browse repository at this point
Copy the full SHA d9aecc2View commit details -
Configuration menu - View commit details
-
Copy full SHA for 830de99 - Browse repository at this point
Copy the full SHA 830de99View commit details
Commits on May 21, 2025
-
Configuration menu - View commit details
-
Copy full SHA for 4e51d12 - Browse repository at this point
Copy the full SHA 4e51d12View commit details -
Configuration menu - View commit details
-
Copy full SHA for a335cf6 - Browse repository at this point
Copy the full SHA a335cf6View commit details -
simplify the interface to support non .jl files
The interface added in #680 for supporting non-.jl files assumes usages of custom struct, which requires overloading functions like `abspath` and is therefore quite cumbersome. Worse still, that makes it harder to refactor Revise code base, since now we need to think about the possibility of arbitrary types in places that originally needed to handle `AbstractString`. This commit provides a simpler interface: support custom files by registering parsers per file extension in `Revise.parsers::Dict{String,Any}`. The greatly simplified `non_jl_test.jl` demonstrates how much more straightforward this approach is. Of course, a potential downside is that if multiple packages register different parsers for the same extension, conflicts could arise, but since this feature is experimental, I believe we don’t need to account for that scenario yet.
Configuration menu - View commit details
-
Copy full SHA for 2f11cda - Browse repository at this point
Copy the full SHA 2f11cdaView commit details
Commits on May 22, 2025
-
remove the interface for supporting non-jl files
Remove interface for supporting non-.jl files introduced in #680. This interface forces functions that used to handle only `AbstractString` to deal with arbitrary types, making refactoring across the Revise.jl codebase difficult. The simplified interface proposed in the previous commit is also fragile since it relies on initialization and runtime-mutable data structures, and so it is therefore not a complete solution. After discussion, and since this experimental interface has effectively no users, we remove it for now to enable codebase refactoring until it becomes necessary again.
Configuration menu - View commit details
-
Copy full SHA for f63d897 - Browse repository at this point
Copy the full SHA f63d897View commit details -
Configuration menu - View commit details
-
Copy full SHA for 0ea7485 - Browse repository at this point
Copy the full SHA 0ea7485View commit details -
Merge pull request #921 from timholy/avi/simple-non-jl
remove the interface to support non .jl files
Configuration menu - View commit details
-
Copy full SHA for e47b794 - Browse repository at this point
Copy the full SHA e47b794View commit details
Commits on May 23, 2025
-
Configuration menu - View commit details
-
Copy full SHA for 3ab981a - Browse repository at this point
Copy the full SHA 3ab981aView commit details -
some more minor cleanups on the code base (#922)
Separated from the refactoring I'm working on locally.
Configuration menu - View commit details
-
Copy full SHA for 2911b1e - Browse repository at this point
Copy the full SHA 2911b1eView commit details
Commits on May 27, 2025
-
2
Configuration menu - View commit details
-
Copy full SHA for 92cb6b8 - Browse repository at this point
Copy the full SHA 92cb6b8View commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff v3.7.6...v3.8.0