Skip to content

[ADHOC] Separate required files for installation step for dataflow-gen-worker build - #3215

Open
brandantck wants to merge 5 commits into
developfrom
brandantck/ADHOC-optimize-dataflow-gen-worker-build
Open

[ADHOC] Separate required files for installation step for dataflow-gen-worker build#3215
brandantck wants to merge 5 commits into
developfrom
brandantck/ADHOC-optimize-dataflow-gen-worker-build

Conversation

@brandantck

@brandantck brandantck commented Aug 26, 2026

Copy link
Copy Markdown
Collaborator
  1. Separate required files for installation so that any change to flow code does not trigger the long-running install steps

Merge Checklist

Please cross check this list if additions / modifications needs to be done on top of your core changes and tick them off. Reviewer can as well glance through and help the developer if something is missed out.

  • Automated Tests (Jasmine integration tests, Unit tests, and/or Performance tests)
  • Updated Manual tests / Demo Config
  • Documentation (Application guide, Admin guide, Markdown, Readme and/or Wiki)
  • Verified that local development environment is working with latest changes (integrated with latest develop branch)
  • following best practices in code review doc

Copilot AI balanced review requested due to automatic review settings August 26, 2026 17:11

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Separates dependency inputs from flow source code to preserve expensive Docker build caches.

Changes:

  • Adds a dependency-only staging layer.
  • Defers copying full flow sources until the final image.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@brandantck
brandantck enabled auto-merge September 4, 2026 01:51
@brandantck
brandantck added this pull request to the merge queue Sep 4, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Sep 4, 2026
@brandantck
brandantck added this pull request to the merge queue Sep 4, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Sep 4, 2026
@brandantck
brandantck added this pull request to the merge queue Sep 4, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Sep 4, 2026
@brandantck
brandantck added this pull request to the merge queue Sep 4, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Sep 4, 2026
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.

4 participants