Skip to content

Conversation

Copilot
Copy link

@Copilot Copilot AI commented Aug 31, 2025

  • Replace no-duplicate-imports with import/no-duplicates rule
  • Add eslint-plugin-import to peerDependencies
  • Update eslint configuration to use import plugin
  • Run npm install and update package-lock.json

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copy link
Author

Copilot AI commented Aug 31, 2025

@Apollon77 👋 This repository doesn't have Copilot instructions. With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs.

I can generate a .github/copilot-instructions.md file for you automatically. Click here to open a pre-filled issue and assign it to me. I'll write the instructions, and then tag you for review.

@Copilot Copilot AI changed the title [WIP] Allow separate import lines per import soiurce for types and non types Allow separate import lines for types and non-types from the same module Aug 31, 2025
@Copilot Copilot AI requested a review from Apollon77 August 31, 2025 20:42
Copilot finished work on behalf of Apollon77 August 31, 2025 20:42
@Apollon77
Copy link
Contributor

@copilot Please execute "npm install" and also commit the updated package-lock.json file

Copy link
Author

Copilot AI commented Aug 31, 2025

@copilot Please execute "npm install" and also commit the updated package-lock.json file

Done! I've executed npm install and committed the updated package-lock.json file in commit 3e830d9.

Copilot finished work on behalf of Apollon77 August 31, 2025 21:28
@Apollon77 Apollon77 marked this pull request as ready for review August 31, 2025 21:29
@Apollon77
Copy link
Contributor

I have tested the change manually in matter adapter

@mcm1957
Copy link
Contributor

mcm1957 commented Sep 1, 2025

Does this change allow duplicate imports in general or only one types import and one import without tyoes from same source?

Duplicate imports in general should not be allowed in my opinnion.

@Apollon77
Copy link
Contributor

It allows just one or one type and one non-type.

@Apollon77 Apollon77 added question Further information is requested and removed question Further information is requested labels Sep 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