Skip to content

Conversation

andrewheard
Copy link
Contributor

Created a "fork" of the FirebaseAppCheck directory and podspec as a starting point for a generic (non-Firebase-specific) version of the SDK.

  • Renamed FirebaseAppCheck files and directories to AppCheck.
  • Changed ObjC prefixes from FIR to GAC for App Check files and symbols.
  • Prefixed Swift names with Internal.
  • Copied and renamed AppCheckBackoffWrapperFake and AppCheckFake from SharedTestUtilities to AppCheck/Tests/Utils.

Note: This is purely a copy and rename of FirebaseAppCheck and it still depends on FirebaseCore.

Created a "fork" of the FirebaseAppCheck directory and podspec as a
starting point for a generic (non-Firebase-specific) version of the SDK.

- Renamed FirebaseAppCheck files and directories to AppCheck.
- Changed ObjC prefix from FIR to GAC for App Check files and symbols.
- Prefixed Swift names with "Internal".
- Copied and renamed AppCheckBackoffWrapperFake and AppCheckFake from
  SharedTestUtilities to AppCheck/Test/Utils.

Note: This is purely a copy and rename of FirebaseAppCheck and it still
      depends on FirebaseCore.
@google-oss-bot
Copy link

2 Warnings
⚠️ Did you forget to add a changelog entry? (Add #no-changelog to the PR description to silence this warning.)
⚠️ New public headers were added, did you remember to add them to the umbrella header?

Generated by 🚫 Danger

@andrewheard andrewheard merged commit 80357b4 into appcheck-lite May 9, 2023
@andrewheard andrewheard deleted the ah/appcheck-lite branch May 9, 2023 13:51
andrewheard added a commit that referenced this pull request May 9, 2023
- Renamed additional FIR-prefixed symbols that were missed in PR #11247.
- Removed GACAppCheckComponent since the generic App Check SDK is not
  intended to register with the Firebase components system directly.
- Added `AppCheck` as a dependency of `FirebaseAppCheck`.
@firebase firebase locked and limited conversation to collaborators Jun 9, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants