Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 6, 2025

Updates build configuration to enable code coverage collection and reduce the minimum coverage warning threshold from 70% to 40% across all targets.

Changes

  • Added codecov operation to all three targets (iOS Library, Mac Library, Vision Library)
  • Reduced min_warn_codecov from 70.0 to 40.0 for all targets
target_specifiers = [
    {
        "name" : "iOS Library",
        "target" : "ios_library",
        "scheme" : "IdentityCore iOS",
-       "operations" : [ "build", "test" ],
+       "operations" : [ "build", "test", "codecov" ],
-       "min_warn_codecov" : 70.0,
+       "min_warn_codecov" : 40.0,
        "platform" : "iOS",
    },
    # Similar updates for Mac Library and Vision Library targets
]

The codecov operation triggers code coverage collection during builds (via -enableCodeCoverage YES flag) and generates coverage reports using llvm-cov.

Original prompt

On build.py, In this file, update the operations in all targets to include operation for code coverage. Also reduce the minimum codecov warning threshold to 40 instead of 70 in all targets


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI self-assigned this Nov 6, 2025

target_specifiers = [
{
"name" : "iOS Library",

Choose a reason for hiding this comment

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

This pull request does not update changelog.txt.

Please consider if this change would be noticeable to a partner or user and either update changelog.txt or resolve this conversation.

Copilot AI changed the title [WIP] Update build.py for code coverage operations Add codecov operation and reduce warning threshold to 40% in build.py Nov 6, 2025
Copilot AI requested a review from ameyapat November 6, 2025 03:08
Copilot finished work on behalf of ameyapat November 6, 2025 03:08
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.

2 participants