Skip to content

Fixing redundancy in the airbyte_to_flow step in GitHub Actions#347

Open
Techtacles wants to merge 1 commit intoestuary:masterfrom
Techtacles:emmanuel/fix-github-actions-redundancy
Open

Fixing redundancy in the airbyte_to_flow step in GitHub Actions#347
Techtacles wants to merge 1 commit intoestuary:masterfrom
Techtacles:emmanuel/fix-github-actions-redundancy

Conversation

@Techtacles
Copy link

PR Title

Refactor airbyte_to_flow job to use general conditional (DRY)

Description

Summary

This PR refactors the airbyte_to_flow step in the GitHub Actions pipeline by removing repeated conditional checks.

Details

  • Previously, multiple steps repeated the condition:
    if: needs.paths-filter.outputs.atf_modified == 'true'

To follow the DRY principle, I moved this into a general job-level conditional instead of repeating it at each step.

Benefits

Cleaner and more maintainable workflow file.

Reduces redundancy and potential errors from duplicated conditions.

@Techtacles
Copy link
Author

Please take a look and let me know your thoughts

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