Skip to content

Conversation

@Mansi-mParticle
Copy link
Contributor

Instructions

  1. PR target branch should be against development
  2. PR title name should follow this format: https://github.com/mParticle/mparticle-workflows/blob/main/.github/workflows/pr-title-check.yml
  3. PR branch prefix should follow this format: https://github.com/mParticle/mparticle-workflows/blob/main/.github/workflows/pr-branch-check-name.yml

Summary

  • Updated README.md with consistent formatting. Added CONTRIBUTING.md to define contribution guidelines. Added RELEASING.md to document the release process.To standardize documentation structure with other repositories.

Testing Plan

  • Was this tested locally? If not, explain why.
  • {explain how this has been tested, and what, if any, additional testing should be done}

Reference Issue (For mParticle employees only. Ignore if you are an outside contributor)

* added test for logCommerceEvent when event is nil

* full test coverage for logCommerceEvent

* full test coverage for logEvent

* added test for logCommerceEvent when event is nil

* full test coverage for logCommerceEvent

* full test coverage for logEvent

* result of call with @BrandonStalnaker

* remove unnecessary nil check from logCommerceEvent
* Organize tests into a mark

* added full test coverage for LTVIncrease

* fix misspell of mock parameter

* add comments to make more readable
@Mansi-mParticle Mansi-mParticle requested a review from a team as a code owner September 25, 2025 16:09
Copy link
Contributor

@nickolas-dimitrakas nickolas-dimitrakas left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@rmi22186 rmi22186 left a comment

Choose a reason for hiding this comment

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

Need to wait for https://github.com/mParticle/mparticle-apple-sdk/pull/415/files to be merged before this one is

I'd make sure @BrandonStalnaker , who has done the most contributions, provides commentary/approval for this before merging as well.

Base automatically changed from development to main October 1, 2025 03:19
@rmi22186
Copy link
Member

rmi22186 commented Oct 1, 2025

For some reason UnitTests/MParticleTestsSwift.swift and UnitTests/Mocks/MPDataPlanFilterMock.swift got added to this PR. Shouldn't be.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants