Skip to content

Adding MANIFEST.in file to include package version files in builds#263

Merged
cleemullins merged 6 commits intomainfrom
users/robrandao/package-version
Dec 1, 2025
Merged

Adding MANIFEST.in file to include package version files in builds#263
cleemullins merged 6 commits intomainfrom
users/robrandao/package-version

Conversation

@rodrigobr-msft
Copy link
Contributor

This pull request improves the versioning process for all Python library packages by ensuring that each package reads its version from a VERSION.txt file, which is generated during the build process. This change makes version management more consistent and reliable across all libraries.

Build and versioning improvements:

  • The CI build script (.azdo/ci-pr.yaml) now generates a VERSION.txt file containing the package version for each library before building the packages.
  • All Python library setup.py files have been updated to read the version from VERSION.txt if it exists, falling back to the PackageVersion environment variable otherwise. This ensures the correct version is always used during packaging. [1] [2] [3] [4] [5] [6] [7] [8]

Packaging configuration updates:

  • Each library's MANIFEST.in now includes the VERSION.txt file so that it is packaged and distributed with the built wheel or source distribution.

Copilot AI review requested due to automatic review settings December 1, 2025 17:31
Copy link
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 pull request standardizes version management across all Python libraries by introducing a VERSION.txt file-based approach. The CI build process now generates VERSION.txt files before building packages, and each library's MANIFEST.in ensures this file is included in source distributions.

  • CI build script generates VERSION.txt files containing package versions
  • All setup.py files read version from VERSION.txt with fallback to environment variable
  • MANIFEST.in files added to include VERSION.txt in source distributions

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated no comments.

Show a summary per file
File Description
libraries/microsoft-agents-activity/MANIFEST.in Adds VERSION.txt to source distribution
libraries/microsoft-agents-authentication-msal/MANIFEST.in Adds VERSION.txt to source distribution
libraries/microsoft-agents-copilotstudio-client/MANIFEST.in Adds VERSION.txt to source distribution
libraries/microsoft-agents-hosting-aiohttp/MANIFEST.in Adds VERSION.txt to source distribution
libraries/microsoft-agents-hosting-core/MANIFEST.in Adds VERSION.txt to source distribution
libraries/microsoft-agents-hosting-fastapi/MANIFEST.in Adds VERSION.txt to source distribution
libraries/microsoft-agents-hosting-teams/MANIFEST.in Adds VERSION.txt to source distribution
libraries/microsoft-agents-storage-blob/MANIFEST.in Adds VERSION.txt to source distribution
libraries/microsoft-agents-storage-cosmos/MANIFEST.in Adds VERSION.txt to source distribution

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@cleemullins cleemullins merged commit 319d84e into main Dec 1, 2025
10 checks passed
@cleemullins cleemullins deleted the users/robrandao/package-version branch December 1, 2025 20:31
rodrigobr-msft added a commit that referenced this pull request Dec 1, 2025
)

* Writing version to file and reading it when building

* Fixing issue

* Removed unnecessary declaration

* Specifying encoding

* Adding MANIFEST.in files to include VERSION.txt
rodrigobr-msft added a commit that referenced this pull request Dec 1, 2025
* Getting package version from file when building packages (#261)

* Writing version to file and reading it when building

* Fixing issue

* Removed unnecessary declaration

* Specifying encoding

* Adding MANIFEST.in file to include package version files in builds (#263)

* Writing version to file and reading it when building

* Fixing issue

* Removed unnecessary declaration

* Specifying encoding

* Adding MANIFEST.in files to include VERSION.txt

* Fixing _sign_in_state serialization/deserialization (#264)

* Fixing _sign_in_state (de)serialization

* Reformatting

---------

Co-authored-by: Chris Mullins <[email protected]>

---------

Co-authored-by: Chris Mullins <[email protected]>
Copilot AI mentioned this pull request Dec 2, 2025
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