Skip to content
This repository was archived by the owner on Jun 22, 2026. It is now read-only.

Updated Import collection params#22

Merged
eldonm merged 2 commits into
mainfrom
0.1.7
Aug 29, 2025
Merged

Updated Import collection params#22
eldonm merged 2 commits into
mainfrom
0.1.7

Conversation

@Tharickv75

Copy link
Copy Markdown
Contributor

Type of Change

What type of change does this PR introduce? Mark all that apply:

  • 🐛 Bug Fix
  • 🚀 Feature Request
  • 🔄 Refactor
  • 📖 Documentation Update
  • 🔧 Other (Please specify):

Summary

What does this PR address?

Refactored the import_collection function parameters for clarity and consistency.

  • Replaced collection_info with data
  • Renamed purge_collection to purge

Description

Refactor Request:

  • Area Refactored: import_collection function signature.
  • Problem: The parameter names were unnecessarily verbose and inconsistent with naming conventions used elsewhere in the codebase.
  • Improvements: Simplified parameter names to make the function easier to read and maintain.

Changes Made

High-Level Summary:

  1. Updated import_collection parameter collection_info: dictdata: dict.
  2. Updated import_collection parameter purge_collection: boolpurge: bool.
  3. Updated internal function logic to match new parameter names.

Checklist

Mark all that apply:

  • Code follows the project’s coding guidelines.
  • Tests have been added or updated for new functionality.
  • Documentation has been updated (if applicable).
  • Existing tests pass locally with these changes.
  • Any dependencies introduced are justified and documented.

@Tharickv75 Tharickv75 requested a review from eldonm August 27, 2025 20:44
@Tharickv75 Tharickv75 self-assigned this Aug 27, 2025
@eldonm eldonm merged commit 43964e0 into main Aug 29, 2025
2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants