Skip to content

Commit

Permalink
Tidy
Browse files Browse the repository at this point in the history
  • Loading branch information
Ekrekr committed Mar 11, 2024
1 parent c2fd6ca commit 0f59ec8
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 3 deletions.
1 change: 0 additions & 1 deletion core/main.ts
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,6 @@ export function main(coreExecutionRequest: Uint8Array | string): Uint8Array | st

// Read the workflow settings from the root of the project.
let projectConfig = readWorkflowSettings();
console.log("🚀 ~ main ~ projectConfig:", projectConfig);

// Merge in project config overrides.
const projectConfigOverride = compileRequest.compileConfig.projectConfigOverride ?? {};
Expand Down
1 change: 0 additions & 1 deletion core/workflow_settings.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ const nativeRequire = typeof __webpack_require__ === "function" ? __non_webpack_

export function readWorkflowSettings(): dataform.ProjectConfig {
const workflowSettingsYaml = maybeRequire("workflow_settings.yaml");
console.log("🚀 ~ readWorkflowSettings ~ workflowSettingsYaml:", workflowSettingsYaml);
// `dataform.json` is deprecated; new versions of Dataform Core prefer `workflow_settings.yaml`.
const dataformJson = maybeRequire("dataform.json");

Expand Down
3 changes: 2 additions & 1 deletion protos/configs.proto
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,8 @@ message WorkflowSettings {
// Optional. The prefix to append to all action names.
string name_prefix = 9;

// Optional. Options for Notebook actions.
// Optional. Default storage bucket to output notebooks in their
// post-execution state.
string default_notebook_output_bucket = 10;
}

Expand Down

0 comments on commit 0f59ec8

Please sign in to comment.