Skip to content

Comments

Remove multiapi functionality from repository#3188

Merged
iscai-msft merged 6 commits intomainfrom
copilot/fix-3187
Aug 28, 2025
Merged

Remove multiapi functionality from repository#3188
iscai-msft merged 6 commits intomainfrom
copilot/fix-3187

Conversation

Copy link
Contributor

Copilot AI commented Aug 28, 2025

This PR completely removes the multiapi functionality from the autorest.python repository as requested in the issue. The multiapi feature allowed generating Python clients that could handle multiple API versions, but it is no longer needed and has been fully eliminated.

Changes Made

Source Code Removal

  • Removed the entire packages/autorest.python/autorest/multiapi/ directory containing the core multiapi implementation
  • Removed multiapi plugin registration from the JSON-RPC server
  • Removed multiapi configuration flag from the code generation options

Test Infrastructure Removal

  • Removed the entire packages/autorest.python/test/multiapi/ directory containing:
    • 9 different multiapi test specifications
    • Generated test outputs for all multiapi variants
    • Test configuration files (pytest.ini, tox.ini, requirements.txt)

Sample Code Removal

  • Removed the entire packages/autorest.python/samples/specification/multiapi/ directory
  • Updated samples README to remove multiapi references

Documentation Updates

  • Removed docs/generate/multiapi.md - the complete multiapi generation guide
  • Updated docs/generate/readme.md to remove multiapi links
  • Removed multiapi sections from client initialization and models documentation

Configuration Updates

  • Removed multiapi pipeline configuration from main README.md
  • Updated build tasks to remove multiapi generation functions
  • Updated CI pipeline to remove multiapi test execution
  • Cleaned up spell checker dictionary to remove multiapi-related words

Build System Updates

  • Removed multiapi references from utility scripts
  • Updated task definitions to exclude multiapi operations

Verification

All modified files pass syntax validation, and the repository structure is clean with no remaining multiapi directories or broken imports. The removal is comprehensive and maintains the integrity of the remaining codebase.

Fixes #3187.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • artprodcus3.artifacts.visualstudio.com
    • Triggering command: node /home/REDACTED/.local/share/pnpm/.tools/pnpm/9.5.0/bin/pnpm install (dns block)

If you need me to access, download, or install something from one of these locations, you can either:


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

Co-authored-by: msyyc <70930885+msyyc@users.noreply.github.com>
Copilot AI changed the title [WIP] remove "multiapi" from repo Remove multiapi functionality from repository Aug 28, 2025
Copilot AI requested a review from msyyc August 28, 2025 01:14
@msyyc msyyc marked this pull request as ready for review August 28, 2025 09:48
@msyyc msyyc force-pushed the copilot/fix-3187 branch from 680ac93 to ade0633 Compare August 28, 2025 10:08
Copy link
Contributor

@iscai-msft iscai-msft left a comment

Choose a reason for hiding this comment

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

@copilot you can assume all code that refers to code_model.options["multiapi"] is false and clean up this code

@iscai-msft iscai-msft merged commit 22789e5 into main Aug 28, 2025
15 checks passed
@iscai-msft iscai-msft deleted the copilot/fix-3187 branch August 28, 2025 20:35
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.

remove "multiapi" from repo

3 participants