Skip to content
This repository was archived by the owner on Apr 15, 2024. It is now read-only.

Fab 6046: Add Pipeline generation and repository configuration based on workspaces #745

Draft
wants to merge 20 commits into
base: develop
Choose a base branch
from
Draft

Fab 6046: Add Pipeline generation and repository configuration based on workspaces #745

wants to merge 20 commits into from

Conversation

@laguirre-cs
Copy link
Contributor

@laguirre-cs laguirre-cs commented Mar 27, 2024

Checklist:

Please check you fulfill ALL of the relevant checkboxes

  • Notified docs of any potential USER-facing changes
  • Added short description of the change - with relevant motivation and context.
  • Branch has the ticket number in its name (along with a ticket summary)
  • Commented the code, particularly in hard-to-understand areas
  • Added tests that prove my fix is effective or that my feature works
  • Ran npm test and it passes
  • Changes generate no new warnings
  • Changes are backward compatible with older clusters

Still a work in progress

@laguirre-cs laguirre-cs force-pushed the FAB-6046-cli-generate branch from ff399cb to 9e01c19 Compare April 2, 2024 18:31
@laguirre-cs
Add minimal implementation for Pipelines configure & generate CLI com…
3af6478 
…mands based on workspaces by using inheritance - avoiding conflicts in Config file
@laguirre-cs
WIP - Refactoring BaseGenerateCommand (trying to be more readable). F…
ee811ac 
…ix bug when looking up config section
@laguirre-cs
Add comments to BaseWorkspaceGenerate command
c743870
@laguirre-cs
Fix typo in README
3a54bdb
@laguirre-cs
Reorganize WorkspaceConfigureCommand to have a shared super class wit…
8cf915b 
…h PipelinesConfigureCommand
@laguirre-cs
Rename base template configure/generate commands. Refactor generation…
73b7936 
… logic into smaller functions with a custom type
@laguirre-cs
Add missing await that caused no file tree to be returned
1079cd5
@laguirre-cs
Add variables for ANSI escape sequences to reduce clutter
2406eda
@laguirre-cs
Use printToTerminal() instead of console.log() for all output
2158b0a
@laguirre-cs
Apply enabled & resourceType filters for templates matching the speci…
ad754c0 
…fied metadata.json
@laguirre-cs
Add temporary docs folder - maybe not appropriate for public repo
4182182
@laguirre-cs
Add TOOD and note about templating
7d48f87
@laguirre-cs
Add debugging logs to the Generate command, and add ability to filter…
36f031c 
… filenames for Pipelines - dbt (jinja2) clashes with lodash
@laguirre-cs
Rename method name and update coumment
295232d
@laguirre-cs
Remove usage of printError() in new CLI commands and wrap with callCo…
08a3c28 
…mmand()
@laguirre-cs
Add color & profile flags to Pipeline generate & configure CLI subcom…
423be0c 
…mands. Add validation for Pipeline Names - must match pre-configure regex
@laguirre-cs
Fix bug when checking for Empty strings - value might be null/undefined
3ceeb24
@laguirre-cs
Fix typo in --notree option description
e884248
@laguirre-cs
Fix typo in dbt_project name
4a87188
@laguirre-cs laguirre-cs force-pushed the FAB-6046-cli-generate branch from fcafe22 to 4a87188 Compare April 10, 2024 14:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@laguirre-cs