Skip to content

Automatic reservation application#26

Merged
max-ostapenko merged 20 commits intomainfrom
auto-apply
Jan 20, 2026
Merged

Automatic reservation application#26
max-ostapenko merged 20 commits intomainfrom
auto-apply

Conversation

@max-ostapenko
Copy link
Contributor

@max-ostapenko max-ostapenko commented Jan 15, 2026

This pull request introduces a new "automatic reservation" feature that allows reservation settings to be applied globally across all Dataform actions, reducing the need for repetitive, manual reservation code in each definition. The update includes helper functions, documentation, and test project changes to demonstrate and validate this new approach.

Key changes include:

New Feature: Automatic Reservation Application

  • Added applyAutomaticReservations and supporting helpers to index.js, enabling reservation settings to be automatically applied to all relevant actions by monkeypatching global Dataform methods. This removes the need for per-action or per-file reservation boilerplate.

Documentation Updates

  • Updated README.md to document both manual and new automatic reservation application approaches, including code examples for each.

Test Project Refactor

  • Removed manual reservation application code (pre_operations blocks and .preOps calls) from all test definitions, relying instead on the new automatic reservation mechanism.
  • Added a new test definition (test_automation.js) that demonstrates usage of applyAutomaticReservations in a project context.

…oving manual `pre_operations` from definitions.
Signed-off-by: Max Ostapenko <1611259+max-ostapenko@users.noreply.github.com>
…m action types, introduce comprehensive testing

Signed-off-by: Max Ostapenko <1611259+max-ostapenko@users.noreply.github.com>
Signed-off-by: Max Ostapenko <1611259+max-ostapenko@users.noreply.github.com>
Signed-off-by: Max Ostapenko <1611259+max-ostapenko@users.noreply.github.com>
Signed-off-by: Max Ostapenko <1611259+max-ostapenko@users.noreply.github.com>
…d file, and update ESLint globals.

Signed-off-by: Max Ostapenko <1611259+max-ostapenko@users.noreply.github.com>
Signed-off-by: Max Ostapenko <1611259+max-ostapenko@users.noreply.github.com>
…t string or array pre-operations

Signed-off-by: Max Ostapenko <1611259+max-ostapenko@users.noreply.github.com>
…n tests

Signed-off-by: Max Ostapenko <1611259+max-ostapenko@users.noreply.github.com>
…r automated testing and assignment improvements
…nd update workflow_settings.yaml to uncomment defaultDataset
@max-ostapenko max-ostapenko merged commit cec4ae1 into main Jan 20, 2026
6 checks passed
@max-ostapenko max-ostapenko deleted the auto-apply branch January 20, 2026 13: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.

1 participant