Skip to content

inferedge-moss==1.0.0b18#37

Open
HarshaNalluru wants to merge 1 commit intomainfrom
harshan/b18
Open

inferedge-moss==1.0.0b18#37
HarshaNalluru wants to merge 1 commit intomainfrom
harshan/b18

Conversation

@HarshaNalluru
Copy link
Contributor

Updated the inferedge-moss package version from >=1.0.0b17 to ==1.0.0b18

Copilot AI review requested due to automatic review settings March 11, 2026 20:13
@vercel
Copy link
Contributor

vercel bot commented Mar 11, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
moss-samples Error Error Mar 11, 2026 8:13pm

Request Review

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

Updates the Python examples’ dependency on inferedge-moss to a newer beta version, affecting how users install and run the Python example scripts.

Changes:

  • Changed inferedge-moss in the Python examples requirements from >=1.0.0b17 to ==1.0.0b18.

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

@@ -1,3 +1,3 @@
inferedge-moss>=1.0.0b17
inferedge-moss==1.0.0b18
Copy link

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

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

Pinning inferedge-moss to an exact beta (==1.0.0b18) is inconsistent with the rest of the repo’s Python dependency specs (which use lower bounds like >=...) and can make the example environment unnecessarily brittle (e.g., prevents installing b19/patches). If the goal is just to require features from b18, prefer >=1.0.0b18 (optionally with an upper bound) instead of == unless there is a documented reason to hard-pin.

Suggested change
inferedge-moss==1.0.0b18
inferedge-moss>=1.0.0b18

Copilot uses AI. Check for mistakes.
@@ -1,3 +1,3 @@
inferedge-moss>=1.0.0b17
inferedge-moss==1.0.0b18
Copy link

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

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

The PR description says the inferedge-moss version was updated repo-wide, but there are still references to inferedge-moss>=1.0.0b17 elsewhere (e.g., README.md:61, examples/python/README.md:62, apps/docker/python/requirements.txt:1). If this PR is intended to standardize on b18, those should be updated too; otherwise the PR description should be narrowed to indicate it only changes the Python example requirements file.

Copilot uses AI. Check for mistakes.
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