Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix code review #13

Merged
merged 21 commits into from
May 4, 2023
Merged

Fix code review #13

merged 21 commits into from
May 4, 2023

Conversation

thtri
Copy link
Collaborator

@thtri thtri commented May 4, 2023

Changes

  • Tests
  • Documentation

thtri and others added 21 commits April 6, 2023 17:47
Merge dev to release - 21/04/23
* feat(gradle) support task list

* Change parameter name to buildFlags to align with other piper steps'
…ands (SAP#4310)

* adding a timeout for helm test

* extending test cases

* Upadate the helmTestWaitSeconds parameter

* Add timeout parameter for helm test command

* Update tests

---------

Co-authored-by: Vyacheslav Starostin <[email protected]>
* CodeQL compliance report and check

* fix test cases

---------

Co-authored-by: Daria Kuznetsova <[email protected]>
* DESS-1710: modified WaitForBuildToFinish function - added exception handling

* DESS-1710: fixed parameters in WaitForBuildToFinish

* DESS-1710: added error handling in WaitForBuildToFinish

* DESS-1710: formatted build.go

* DESS-1710: fixed error logging

* DESS-1710: fixed Poll retrying

* DESS-1710: renamed WaitForBuildToFinish

* DESS-1710: refactored WaitForBuildToFinish

* DESS-1710: changed maxRetries to 4 attempt

* DESS-1710: fixed error handling

---------

Co-authored-by: Christopher Fenner <[email protected]>
Co-authored-by: Ashly Mathew <[email protected]>
@thtri thtri merged commit c87df60 into cxone-dev May 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants