Skip to content

Conversation

@DavidS-ovm
Copy link
Contributor

@DavidS-ovm DavidS-ovm commented Aug 3, 2023

Fixes: #32

The change has to be in DEFINING and have the same ticket link to be
eligible for re-use.

This will reduce the amount of abandoned Changes when using the github
action.
@DavidS-ovm DavidS-ovm force-pushed the list-changes-by-status branch from e0c8c20 to 566b5da Compare August 3, 2023 09:41
})
if err != nil {
log.WithContext(ctx).WithError(err).WithFields(lf).Error("failed to create change")
log.WithContext(ctx).WithError(err).WithFields(lf).Error("failed to searching for existing changes")

Check failure

Code scanning / CodeQL

Clear-text logging of sensitive information

[Sensitive data returned by an access to apiKey](1) flows to a logging call.
},
})
if err != nil {
log.WithContext(ctx).WithError(err).WithFields(lf).Error("failed to create change")

Check failure

Code scanning / CodeQL

Clear-text logging of sensitive information

[Sensitive data returned by an access to apiKey](1) flows to a logging call.

changeUUID = createResponse.Msg.Change.Metadata.GetUUIDParsed()
if changeUUID == nil {
log.WithContext(ctx).WithError(err).WithFields(lf).Error("failed to read change id")

Check failure

Code scanning / CodeQL

Clear-text logging of sensitive information

[Sensitive data returned by an access to apiKey](1) flows to a logging call.
@DavidS-ovm DavidS-ovm merged commit 57ea657 into main Aug 3, 2023
@DavidS-ovm DavidS-ovm deleted the list-changes-by-status branch August 3, 2023 10:30
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.

Allow the CLI to update changes rather than create new ones

3 participants