Skip to content

Remove obsolete AOT CI parameters from planetarycomputer ci.yml#56238

Merged
chahibi merged 1 commit intoAzure:mainfrom
chahibi:mpc-ci-aot
Feb 13, 2026
Merged

Remove obsolete AOT CI parameters from planetarycomputer ci.yml#56238
chahibi merged 1 commit intoAzure:mainfrom
chahibi:mpc-ci-aot

Conversation

@chahibi
Copy link
Copy Markdown
Member

@chahibi chahibi commented Feb 12, 2026

Remove obsolete AOT CI parameters from planetarycomputer ci.yml

Description

Removes the CheckAOTCompat and AOTTestInputs parameters from sdk/planetarycomputer/ci.yml to align with the changes introduced in PR #53752.

PR #53752 simplified the AOT compatibility pipeline by consolidating AOT checking logic into the build scripts themselves, eliminating the need for explicit CI parameters. However, the planetarycomputer service directory was not included in that change.

Changes

  • Removed CheckAOTCompat: true from sdk/planetarycomputer/ci.yml
  • Removed AOTTestInputs block from sdk/planetarycomputer/ci.yml

Why

These parameters are no longer consumed by the pipeline templates. AOT compatibility is now automatically determined by:

  • The AotCompatOptOut MSBuild property in project files
  • The pipeline template iterating over Artifacts directly instead of requiring separate AOTTestInputs

@chahibi chahibi requested a review from jsquire as a code owner February 12, 2026 23:11
Copilot AI review requested due to automatic review settings February 12, 2026 23:11
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes obsolete AOT compatibility CI parameters (CheckAOTCompat and AOTTestInputs) from the planetarycomputer service directory's ci.yml file. These parameters were made obsolete by PR #53752, which consolidated AOT compatibility checking logic directly into the build scripts. The pipeline now automatically determines AOT compatibility requirements by checking the AotCompatOptOut MSBuild property in project files and iterating over the Artifacts list directly, eliminating the need for separate CI configuration parameters.

Changes:

  • Removed CheckAOTCompat: true parameter from sdk/planetarycomputer/ci.yml
  • Removed AOTTestInputs configuration block that specified the ExpectedAotWarnings.txt filepath

@chahibi chahibi merged commit 5e9cdff into Azure:main Feb 13, 2026
30 checks passed
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.

3 participants