Skip to content
This repository has been 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

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
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
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.
1 participant
@laguirre-cs