Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move Item Functions to Item Module #38

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

JacobKnightley
Copy link
Collaborator

This pull request includes significant changes to the fabric_cicd module to improve error handling, file processing, and the publishing workflow for different item types. The most important changes include modifying error handling in the invoke method, adding a new ItemFile class for file processing, and updating the publishing methods for data pipelines, notebooks, and reports.

Error Handling Improvements:

New File Processing Class:

Publishing Workflow Enhancements:

Codebase Simplification:

@Copilot Copilot bot review requested due to automatic review settings January 18, 2025 00:06
@JacobKnightley JacobKnightley requested a review from a team as a code owner January 18, 2025 00:06

Choose a reason for hiding this comment

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

Copilot reviewed 5 out of 6 changed files in this pull request and generated no comments.

Files not reviewed (1)
  • src/fabric_cicd/fabric_workspace.py: Evaluated as low risk
@JacobKnightley JacobKnightley marked this pull request as draft January 18, 2025 00:07
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.

1 participant