cli icon indicating copy to clipboard operation
cli copied to clipboard

Upload local libraries even if they don't have artifact defined

Open andrewnester opened this issue 1 year ago • 1 comments

Changes

Previously for all the libraries referenced in configuration DABs made sure that there is corresponding artifact section. But this is not really necessary and flexible, because local libraries might be built outside of dabs context. It also created difficult to follow logic in code where we back referenced libraries to artifacts which was difficult to fllow

This PR does 3 things:

  1. Allows all local libraries referenced in DABs config to be uploaded to remote
  2. Simplifies upload and glob references expand logic by doing this in single place
  3. Speed things up by uploading library only once and doing this in parallel

Tests

Added unit + integration tests + made sure that change is backward compatible (no changes in existing tests)

andrewnester avatar Aug 08 '24 10:08 andrewnester

All integration test passed

andrewnester avatar Aug 08 '24 13:08 andrewnester