modern-data-warehouse-dataops icon indicating copy to clipboard operation
modern-data-warehouse-dataops copied to clipboard

Update generic git sample with simplified approach

Open elenaterenzi opened this issue 9 months ago • 2 comments

Type of PR

  • Documentation changes
  • Code changes

Purpose

  • adds a simplified approach that is capable of pulling/pushing changes from/to Fabric to/from the local branch
  • improvements with the comprehensive approach which has been simplified and reuses some of the new scripts introduced with the simplified approach

Does this introduce a breaking change? If yes, details on what can break

No breaking changes introduced

Author pre-publish checklist

  • [ ] Added test to prove my fix is effective or new feature works
  • [x] No PII in logs
  • [x] Made corresponding changes to the documentation

Validation steps

  • Review readme and test the scripts

Issues Closed or Referenced

  • Closes #issue_number
  • References #issue_number

elenaterenzi avatar May 07 '25 08:05 elenaterenzi

@elenaterenzi Do you have a script for "update_from_ws_to_git" in bash? I see all the others in bash and powershell.

good catch @bsherwin , I should have not committed the bash script in the first place as I don't have the _to_git script. We have two options IMO:

  • I remove the bash script, and when I have time to develop and test the other script I will create another PR
  • we wait until I have time to develop the other script (could take a month) let me know your preference

elenaterenzi avatar Jun 26 '25 06:06 elenaterenzi

@elenaterenzi - revisiting this PR. Were you able to find time to create the other bash script? If not, should we just remove the bash and save that for later?

bsherwin avatar Sep 04 '25 13:09 bsherwin