Conversation
* [amsdk-9723] Adding SwiftLint configuration for CI use which sets autocorrectable rules to trigger errors instead of warnings. * [amsdk-9723] [amsdk-9723] Add CircleCI steps to install SwiftLint and lint source code * [amsdk-9723] Change function_body_length in .swiftlint-ci.yml to match recent change in .swiftlint.yml * [amsdk-9723] Change run script for 'unitTests' target to use a single 'swiftlint' call with multiple paths. * [amsdk-9723] Fix errors in .circleci/config.yml and add Swiftlint targets to Makefile. * [amsdk-9723] Correct 'lint' step in .circleci/config.yml and disable Homebrew autoupdate in Makefile * [amsdk-9723] Correct Makefile target names to use '-' instead of '_' * [amsdk-9723] Autocorrect format + fix one formatting error using SwiftLint autocorrect * [amsdk-9723] remove CI specific SwiftLint configuration file. * [amsdk-9723] Move SwiftLint config file to root directory
git-subtree-dir: aepsdk-core-ios git-subtree-split: f525ffd130f0063071f60784b34fbb535052ae01
Make APIs compatible with ObjC
Integrate with Core SPM
…ponse handler changes
Cleanup documentation, formatting
…jects and scripts.
…ipts, and config files.
Rename 'code/unitTests' to 'Tests/UnitTests', 'code/functionalTests' to 'Tests/FunctionalTests', and 'code/testUtilsShared' to 'Tests/TestUtils'. Rename Xcode test targets and schemes to UnitTests and FunctionalTests. Update scripts and Podfile with the renames.
…it doesn't conform with new folder names.
[AMSDK-10442] Cleanup project structure part2
* [AMSDK-10030] Change constants to enums, uppercase * [AMSDK-10030] Change test constants to enums, uppercase * [AMSDK-10030] Fix lint warnings * [AMSDK-10030] Fix lint warnings, count usages, exclude pod dir from lint * [AMSDK-10030] Add name rule exclusion for test files * [AMSDK-10030] Resolve force unwrapps in tests, use unregisterExtension * [AMSDK-10030] Rename ExperiencePlatformContants to Constants after merge conflicts * [AMSDK-10030] Change class to enum for static usage * [DEV] Add task link for open todo * [AMSDK-10030] Review impl - constants reshuffle
[DOC] Include link to public docs
Cleanup & demo app setup
[amsdk-10442] Update Podspec with updated Sources directory.
Skip test IdentityStateFunctionalTests.testSendEvent_withNoECIDInIdentityState_requestSentWithoutECID.
[DEV] Skip test causing intermittent failures in CI.
* Griffon -> Assurance * Use new non-staging build for AEPAssurance * Add deeplink support * Remove internal source for assurance * Fix constants * PR comments * remove bridging header, and refresh pods * update deep link name Co-authored-by: Pravin <pprakash@adobe.com>
* First pass on rebranding to AEPEdge * Minor refactor related to branding * Run lint-autocorrect * Get sample app building and remove dangling files * PR feedback * Update team name in podspec * Update product names * experiencePlatform.configId -> edge.configId * Rename experience event type to edge * Rename test constants * Fix references to product name
Update version to 1.0.0-beta-1
nporter-adbe
approved these changes
Oct 16, 2020
kevinlind
approved these changes
Oct 16, 2020
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: