Skip to content

Conversation

@itsjustmithun
Copy link
Collaborator

No description provided.


from salt_docs.config import load_config, save_config, CONFIG_FILE
from salt_docs.defaults import DEFAULT_CONFIG
from wikigen.config import load_config, save_config, CONFIG_FILE
from salt_docs.mcp.embeddings import get_embedding, get_embeddings_batch
from wikigen.mcp.search_index import FileIndexer
from wikigen.mcp.chunking import chunk_markdown
from wikigen.mcp.embeddings import get_embedding, get_embeddings_batch
@salt-apps
Copy link

salt-apps bot commented Dec 27, 2025

I've successfully fixed the unused import issue!

  • Create fix branch
  • Remove unused import
  • Create fix PR

Created PR #4 which removes the unused CONFIG_FILE import from tests/mode/test_documentation_mode.py. The test still works correctly since it patches the module attribute directly.

@salt-apps
Copy link

salt-apps bot commented Dec 27, 2025

I've successfully fixed the unused import issue.

Todo:

  • Create new branch for the fix
  • Remove unused get_embedding from the import in tests/mcp/test_semantic_search.py
  • Create pull request with the fix

Fix PR created: #5

@itsjustmithun itsjustmithun merged commit a9332c3 into main Dec 27, 2025
7 checks passed
@itsjustmithun itsjustmithun deleted the rename/wikigen branch December 27, 2025 21:23
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.

2 participants