Skip to content
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.

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

Preprocessor to make select constructs public #3880

Merged
merged 9 commits into from
May 9, 2024

Conversation

jamesrb1
Copy link
Contributor

@jamesrb1 jamesrb1 commented May 8, 2024

This PR allows the Paywalls Tester app to access some items internal to purchases-ios by temporarily marking them public during complication.

Items that need to be public have been annotated with \\@PublicForExternalTesting.

The Paywalls Tester project runs a script, Preprocessor.sh, as a pre-action to add public to all classes/structs/enums/functions/inits that have been annotated.

After compilation it runs a second script, Postprocessor.sh, as a post-action to undo these changes so that they don't accidentally get checked in.

The first script is run as a scheme pre-action rather than as a run script build phase, because when run as a build phase the changes to the files aren't picked up until the next compilation attempt.

image

It also changes the archive step to be built with a release configuration, and removes the use of @testable import for debug builds.

resolves PWL-459

jamesrb1 added 2 commits May 8, 2024 13:23
These are run as pre-actions within schemes because adding it has a build phase does not have its changes included in the current build.
@RevenueCat-Danger-Bot
Copy link

RevenueCat-Danger-Bot commented May 8, 2024

1 Message
📖 Size increase: 7.76 KB

Generated by 🚫 Danger

@jamesrb1 jamesrb1 added the build label May 8, 2024
@jamesrb1 jamesrb1 requested a review from a team May 8, 2024 23:10
@jamesrb1 jamesrb1 marked this pull request as ready for review May 8, 2024 23:10
Copy link
Contributor

@tonidero tonidero left a comment

Choose a reason for hiding this comment

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

LGTM! Left some comments but are not blockers.

@jamesrb1 jamesrb1 merged commit 62a8c42 into main May 9, 2024
24 checks passed
@jamesrb1 jamesrb1 deleted the james/paywalls-tester/app-store branch May 9, 2024 18:37
vegaro pushed a commit that referenced this pull request Jun 3, 2024
**This is an automatic release.**

### New Features
* RemoteImage Low Res Image support (#3906) via James Borthwick
(@jamesrb1)
### Bugfixes
* [EXTERNAL] Hide decorative Paywall images from accessibility (#3886)
via @shiftingsand (#3892) via Toni Rico (@tonidero)
### Dependency Updates
* Bump rexml from 3.2.6 to 3.2.8 in
/Tests/InstallationTests/CocoapodsInstallation (#3908) via
dependabot[bot] (@dependabot[bot])
* Bump fastlane-plugin-revenuecat_internal from `dd5e21f` to `8ec0072`
(#3873) via dependabot[bot] (@dependabot[bot])
### Other Changes
* Revert to use xcode 14 to fix deploys (#3929) via Cesar de la Vega
(@vegaro)
* SPMInstallation tests deployment version increase (#3922) via Cesar de
la Vega (@vegaro)
* Only install `swiftlint` on Xcode 15 jobs (#3913) via Josh Holtz
(@joshdholtz)
* Add `http_request_performed` diagnostics event (#3897) via Toni Rico
(@tonidero)
* Paywalls Tester: App Store Prep (#3889) via James Borthwick
(@jamesrb1)
* Paywalls Tester: Enable Example Paywalls in Release Mode (#3885) via
James Borthwick (@jamesrb1)
* Use Ruby 3.2 on CircleCI (#3896) via Josh Holtz (@joshdholtz)
* PaywallsTester: Remove unused code (#3884) via James Borthwick
(@jamesrb1)
* Improved Error Handling (#3883) via James Borthwick (@jamesrb1)
* Clear diagnostics file if we detect it's too big during event tracking
(#3824) via Toni Rico (@tonidero)
* Preprocessor to make select constructs public (#3880) via James
Borthwick (@jamesrb1)
* Paywalls Tester: Use key defined in CI (#3869) via James Borthwick
(@jamesrb1)
* Cleanup: Remove test storekit configuration files when importing
through SPM (#3878) via Andy Boedo (@aboedo)
* Update fastlane plugin and fix docs index path (#3872) via Toni Rico
(@tonidero)
* Update to use xcode 15.3 in CI (#3870) via Cesar de la Vega (@vegaro)
* Paywalls Tester 0.1 (#3868) via James Borthwick (@jamesrb1)
* Update config.yml for SPM repo copy (#3861) via James Borthwick
(@jamesrb1)
* Change deploy-purchase-tester to use xcode 15 (#3858) via Cesar de la
Vega (@vegaro)
nyeu pushed a commit that referenced this pull request Jun 6, 2024
**This is an automatic release.**

### New Features
* RemoteImage Low Res Image support (#3906) via James Borthwick
(@jamesrb1)
### Bugfixes
* [EXTERNAL] Hide decorative Paywall images from accessibility (#3886)
via @shiftingsand (#3892) via Toni Rico (@tonidero)
### Dependency Updates
* Bump rexml from 3.2.6 to 3.2.8 in
/Tests/InstallationTests/CocoapodsInstallation (#3908) via
dependabot[bot] (@dependabot[bot])
* Bump fastlane-plugin-revenuecat_internal from `dd5e21f` to `8ec0072`
(#3873) via dependabot[bot] (@dependabot[bot])
### Other Changes
* Revert to use xcode 14 to fix deploys (#3929) via Cesar de la Vega
(@vegaro)
* SPMInstallation tests deployment version increase (#3922) via Cesar de
la Vega (@vegaro)
* Only install `swiftlint` on Xcode 15 jobs (#3913) via Josh Holtz
(@joshdholtz)
* Add `http_request_performed` diagnostics event (#3897) via Toni Rico
(@tonidero)
* Paywalls Tester: App Store Prep (#3889) via James Borthwick
(@jamesrb1)
* Paywalls Tester: Enable Example Paywalls in Release Mode (#3885) via
James Borthwick (@jamesrb1)
* Use Ruby 3.2 on CircleCI (#3896) via Josh Holtz (@joshdholtz)
* PaywallsTester: Remove unused code (#3884) via James Borthwick
(@jamesrb1)
* Improved Error Handling (#3883) via James Borthwick (@jamesrb1)
* Clear diagnostics file if we detect it's too big during event tracking
(#3824) via Toni Rico (@tonidero)
* Preprocessor to make select constructs public (#3880) via James
Borthwick (@jamesrb1)
* Paywalls Tester: Use key defined in CI (#3869) via James Borthwick
(@jamesrb1)
* Cleanup: Remove test storekit configuration files when importing
through SPM (#3878) via Andy Boedo (@aboedo)
* Update fastlane plugin and fix docs index path (#3872) via Toni Rico
(@tonidero)
* Update to use xcode 15.3 in CI (#3870) via Cesar de la Vega (@vegaro)
* Paywalls Tester 0.1 (#3868) via James Borthwick (@jamesrb1)
* Update config.yml for SPM repo copy (#3861) via James Borthwick
(@jamesrb1)
* Change deploy-purchase-tester to use xcode 15 (#3858) via Cesar de la
Vega (@vegaro)
jamesrb1 pushed a commit that referenced this pull request Jun 19, 2024
**This is an automatic release.**

### New Features
* RemoteImage Low Res Image support (#3906) via James Borthwick
(@jamesrb1)
### Bugfixes
* [EXTERNAL] Hide decorative Paywall images from accessibility (#3886)
via @shiftingsand (#3892) via Toni Rico (@tonidero)
### Dependency Updates
* Bump rexml from 3.2.6 to 3.2.8 in
/Tests/InstallationTests/CocoapodsInstallation (#3908) via
dependabot[bot] (@dependabot[bot])
* Bump fastlane-plugin-revenuecat_internal from `dd5e21f` to `8ec0072`
(#3873) via dependabot[bot] (@dependabot[bot])
### Other Changes
* Revert to use xcode 14 to fix deploys (#3929) via Cesar de la Vega
(@vegaro)
* SPMInstallation tests deployment version increase (#3922) via Cesar de
la Vega (@vegaro)
* Only install `swiftlint` on Xcode 15 jobs (#3913) via Josh Holtz
(@joshdholtz)
* Add `http_request_performed` diagnostics event (#3897) via Toni Rico
(@tonidero)
* Paywalls Tester: App Store Prep (#3889) via James Borthwick
(@jamesrb1)
* Paywalls Tester: Enable Example Paywalls in Release Mode (#3885) via
James Borthwick (@jamesrb1)
* Use Ruby 3.2 on CircleCI (#3896) via Josh Holtz (@joshdholtz)
* PaywallsTester: Remove unused code (#3884) via James Borthwick
(@jamesrb1)
* Improved Error Handling (#3883) via James Borthwick (@jamesrb1)
* Clear diagnostics file if we detect it's too big during event tracking
(#3824) via Toni Rico (@tonidero)
* Preprocessor to make select constructs public (#3880) via James
Borthwick (@jamesrb1)
* Paywalls Tester: Use key defined in CI (#3869) via James Borthwick
(@jamesrb1)
* Cleanup: Remove test storekit configuration files when importing
through SPM (#3878) via Andy Boedo (@aboedo)
* Update fastlane plugin and fix docs index path (#3872) via Toni Rico
(@tonidero)
* Update to use xcode 15.3 in CI (#3870) via Cesar de la Vega (@vegaro)
* Paywalls Tester 0.1 (#3868) via James Borthwick (@jamesrb1)
* Update config.yml for SPM repo copy (#3861) via James Borthwick
(@jamesrb1)
* Change deploy-purchase-tester to use xcode 15 (#3858) via Cesar de la
Vega (@vegaro)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants