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

Cxone release #50

Merged
merged 271 commits into from
Oct 8, 2024
Merged

Cxone release #50

merged 271 commits into from
Oct 8, 2024

Conversation

michaelkubiaczyk
Copy link
Collaborator

Changes

  • Tests
  • Documentation

michaelkubiaczyk and others added 30 commits January 4, 2023 06:45
Merge Michael's branch to cxone-dev
Adding support for CheckmarxOne
DanielMieg and others added 29 commits August 29, 2024 10:55
…SAP#5031)

* Added pagination logic for retrieving projects from Black Duck server

* fixed unit tests for getProject in blackduck

* fixed unit tests for getProject in blackduck

* fixed unit tests for getProject in blackduck
…AP#5032)

* trust engine config and handelling for vault

* add function for resolving trust engine reference

* refactor

* add basic test

* adapt to new trust engine response format

* remove accidental cyclic dependency

* move trust engine hook config

* refactor by separating code from vault

* move trust engine files to own pkg

* adapt to changes of previous commit

* log full error response of trust engine API

* enable getting multiple tokens from trustengine

* remove comment

* incorporate review comments

* go generate

* update unit tests

* apply suggested changes from code review

* fix unit tests

* add unit tests for config pkg

* make changes based on review comments

* make trust engine token available in GeneralConfig and minor fixes

* fix error logic when reading trust engine hook

* make getResponse more flexible and update logging

* update resource reference format

* improve URL handling

* improve logging

* use errors.Wrap() instead of errors.Join()

* update log messages based on suggestions

* remove trustengine resource ref from Sonar step

---------

Co-authored-by: Keshav <[email protected]>
Co-authored-by: jliempt <>
…AP#5037)

* added quotes for mvn settings path

* added logs

* removed logs, added excape symbol for spaces

* set quotes

* removed replacing

* changed quotes

* fixed tests

* removed extra log

---------

Co-authored-by: sumeet patil <[email protected]>
* wildCard MAXX in addon.yml for AAKaaS steps
---------

Co-authored-by: Christian Luttenberger <[email protected]>
Updated CONTRIBUTING.md with Code of Conduct and Guidelines for GenAI and general content restructuring
As we use tags for the unit test files, they don't run if one does not provide the tags=unit flag to the test execution command. This change adds the description of how one is to do this when running the tests in a shell, as well as how to add it to the VS Code extension.

Co-authored-by: tiloKo <[email protected]>
…ation (SAP#5038)

* add trust engine resource reference to parameter docs generation

* test with added resource ref

* remove test resource ref

* use trustengineSecret const

---------

Co-authored-by: jliempt <>
* feat: fail if script is not found

* fix tests

* Fix integration tests

---------

Co-authored-by: Oliver Feldmann <[email protected]>
The cloud sdk pipeline does not exist anymore.
* fix(docs): remove dead links

* Update parameters_test.go
* Add -ws=false -iwr to npm config get registry

* Add -ws=false -iwr to npm config set registry

* Fix test

* Fix test

---------

Co-authored-by: Manjunath <[email protected]>
* Adapting unit test to run for packages with namespace

* Correct type mismatch for object count
* adding anonymisation for object field

* Further fixes to gcts deploy step

* syntax changes
* Run makeBOM after aggregateBOM

* Fix tests
@michaelkubiaczyk michaelkubiaczyk merged commit d8622e6 into cxone-dev-michael Oct 8, 2024
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.