Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 30, 2026

Pylint fails to resolve imports when local dependencies are installed with -e (editable mode) in dev_requirements.txt, causing false positive import-error warnings.

Changes

Removed -e prefix from local package dependencies in dev_requirements.txt files:

- -e ../../core/azure-core
+ ../../core/azure-core
- -e ../../identity/azure-identity  
+ ../../identity/azure-identity

Preserved -e for:

  • azure-sdk-tools (requires editable mode)
  • tests/* paths (test utilities)

Scope

  • 125 dev_requirements.txt files modified
  • 217 editable flags removed

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:

  • scanning-api.github.com
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js (dns block)

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

Original prompt

This section details on the original issue you should resolve

<issue_title>[pylint] remove editable installs</issue_title>
<issue_description>When running pylint locally if a dependency on that package is editably installed in de_reqs we receive (import-error) on all the import statements

ie. from azure.core import ..... (import-error) azure core doesn't exist

to fix this we can remove all the -e from the dev-req installations</issue_description>

<agent_instructions>For all dev_requirements.txt files under sdk/*, remove the -e flag for editable installs strictly based on this criteria:

  • Do NOT remove the flag for requirements under tests/* directories, or for anything with azure-sdk-tools. For example, this should be left alone -e tests/specs_sdk/modeltypes
  • DO remove the flag for anything else, for example -e ../../core/azure-core should not have the -e flag.

DO NOT change any other aspects of the requirements files.</agent_instructions>

Comments on the Issue (you are @copilot in this section)


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

- Removed -e flag from 217 dependency lines across 125 dev_requirements.txt files
- Preserved -e flag for azure-sdk-tools and tests/* paths as required
- No other changes made to requirement files

Co-authored-by: JennyPng <63012604+JennyPng@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove editable installs from dev requirements Remove editable install flags from dev_requirements.txt Jan 30, 2026
Copilot AI requested a review from JennyPng January 30, 2026 21:40
@l0lawrence
Copy link
Member

@scbedd Would we want to do no-ci on this?

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.

[pylint] remove editable installs

3 participants