-
Notifications
You must be signed in to change notification settings - Fork 17
Permalink
Choose a base ref
{{ refName }}
default
Choose a head ref
{{ refName }}
default
Comparing changes
Choose two branches to see what’s changed or to start a new pull request.
If you need to, you can also or
learn more about diff comparisons.
Open a pull request
Create a new pull request by comparing changes across two branches. If you need to, you can also .
Learn more about diff comparisons here.
base repository: getappmap/appmap-python
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v2.1.2
Could not load branches
Nothing to show
Loading
Could not load tags
Nothing to show
{{ refName }}
default
Loading
...
head repository: getappmap/appmap-python
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v2.1.3
Could not load branches
Nothing to show
Loading
Could not load tags
Nothing to show
{{ refName }}
default
Loading
- 8 commits
- 35 files changed
- 2 contributors
Commits on Jul 19, 2024
-
fix: show config packages on startup
In addition to the path of config file, show the packages property, too.
Configuration menu - View commit details
-
Copy full SHA for feec761 - Browse repository at this point
Copy the full SHA feec761View commit details -
fix: add APPMAP_INSTRUMENT_PROPERTIES
Add APPMAP_INSTRUMENT_PROPERTIES to control whether properties should be instrumented.
Configuration menu - View commit details
-
Copy full SHA for 11b6307 - Browse repository at this point
Copy the full SHA 11b6307View commit details
Commits on Jul 26, 2024
-
fix: improve property handling
Base the decision to instrument a property on the property name, rather than trying to use the name of the f{get,set,del} functions. This aligns them with the rest of the class's members, e.g. for exclusion. Also, ensure that those functions only get instrumented once.Configuration menu - View commit details
-
Copy full SHA for 5cce0f0 - Browse repository at this point
Copy the full SHA 5cce0f0View commit details -
fix: try to avoid recording tests
When creating the default config, ignore directories with names that match the regex .*test.*. This should avoid instrumenting the majority of test functions. For those that do still get instrumented, have the test framework integration disable the wrapt wrapped on it, thereby disabling recording.
Configuration menu - View commit details
-
Copy full SHA for 1847b0e - Browse repository at this point
Copy the full SHA 1847b0eView commit details -
Add ruff to dev dependencies, along with an example config. At some point in the future, we may want to switch to using it, rather than pylint.
Configuration menu - View commit details
-
Copy full SHA for ac94204 - Browse repository at this point
Copy the full SHA ac94204View commit details -
The latest update to incremental (which twisted depends on), appears to be broken. Pin to the previous version.
Configuration menu - View commit details
-
Copy full SHA for 7538fa7 - Browse repository at this point
Copy the full SHA 7538fa7View commit details -
Merge pull request #354 from getappmap/no-test-cases_20240715
try to avoid recording tests
Configuration menu - View commit details
-
Copy full SHA for 171625a - Browse repository at this point
Copy the full SHA 171625aView commit details -
Configuration menu - View commit details
-
Copy full SHA for 6fcad4a - Browse repository at this point
Copy the full SHA 6fcad4aView commit details
Loading
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 v2.1.2...v2.1.3