Skip to content

Add base property auto upgrades instrumentation support #24632

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.

Sign up for GitHub

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

Merged
merged 35 commits into from
Apr 25, 2023
Merged

Add base property auto upgrades instrumentation support #24632

Changes from 1 commit
Commits
Show all changes
35 commits
Select commit Hold shift + click to select a range
95d0890
Add initial cross version tests for Property upgrades
asodja Feb 22, 2023
63ccfdb
Initial generation of implementation class for upgraded properties
asodja Feb 28, 2023
b90d0b8
Use Gradle 8.0.2 as a target version in PropertyUpgradesBinaryCompati…
asodja Mar 10, 2023
af7f735
Fix generating setter and add static modifier to generated methods
asodja Mar 10, 2023
52e94a4
Rename generated class for upgraded properties to <original name>_Ada…
asodja Mar 10, 2023
f130070
Small code style polishing of UpgradePropertyInstrumentationExtension
asodja Mar 10, 2023
e644353
Split logic for code generation for property upgrade to multiple classes
asodja Mar 11, 2023
67fb0b5
Rename code-quality-instrumentation to instrumentation-declarations a…
asodja Mar 11, 2023
2f0ddbf
Some more polishing of Property upgrades code generation logic
asodja Mar 11, 2023
1352220
Generate also Groovy interceptor for UpgradedProperty annotation
asodja Mar 13, 2023
1146fe1
Extract original type from generics for Property upgrades
asodja Mar 13, 2023
4508a60
Add com.google.testing.compile:compile-testing dependency
asodja Mar 13, 2023
5fcf370
Add some tests for Property upgrade generated code logic
asodja Mar 13, 2023
fdcb2dc
Fix auto upgrading properties for RegularFileProperty, DirectoryPrope…
asodja Mar 14, 2023
0909a07
Add a test for a Kotlin plugin using Checkstyle with property upgrades
asodja Mar 15, 2023
75b6685
Add NotYetImplemented to PropertyUpgradesBinaryCompatibilityCrossVers…
asodja Apr 4, 2023
e6b13b4
Fix tests after adding property upgrades interceptors
asodja Apr 5, 2023
01fe5b9
Add instrumentation-declarations project to subprojects.json
asodja Apr 5, 2023
4cda300
Fix InternalNullabilityTest
asodja Apr 6, 2023
068246a
Make PropertyUpgradeCodeGenTest work with Java8
asodja Apr 6, 2023
e767e9e
Merge branch 'master' into asodja/property-upgrades-cherry
asodja Apr 11, 2023
1f7ce76
Fixes for property upgrades code gen after merge of master
asodja Apr 11, 2023
f2baa26
Set all dependencies as compileOnly for :instrumentation-declarations
asodja Apr 11, 2023
6aec92c
Small code polish
asodja Apr 11, 2023
70c7b1c
Fix build distribution tests
asodja Apr 11, 2023
915514c
Polish code for UpgradeProperties instrumentation: apply review sugge…
asodja Apr 17, 2023
2fdcf31
Move instumentation generated classes to org.gradle.internal.classpat…
asodja Apr 17, 2023
7616830
Polish code for UpgradeProperties instrumentation: apply additiona re…
asodja Apr 17, 2023
4b9eb52
Rename generated classes for instrumentation and remove stubs
asodja Apr 18, 2023
21dab8b
Polish comments and reduce code duplication for property upgrades ins…
asodja Apr 18, 2023
0ce655b
Additional polishing to property upgrades instrumentation
asodja Apr 20, 2023
7ff8e6e
Generate interceptor declarations with public access
asodja Apr 20, 2023
0e45e93
Use Stream in AbstractInstrumentationProcessor in more cases
asodja Apr 20, 2023
16ee011
Merge branch 'master' into asodja/property-upgrades-cherry
asodja Apr 21, 2023
336cc95
Merge branch 'master' into asodja/property-upgrades-cherry
asodja Apr 25, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge branch 'master' into asodja/property-upgrades-cherry
  • Loading branch information
@asodja
asodja committed Apr 21, 2023
commit 16ee0111ce3d4db872d6b32303ecd9ff3805b6e9

This merge commit was added into this branch cleanly.

There are no new changes to show, but you can still view the diff.