Skip to content

Webhooks: Do not drop partial git info#319

Merged
albertvaka merged 1 commit into
masterfrom
albertvaka/do-not-drop-partial-git-info
Nov 30, 2022
Merged

Webhooks: Do not drop partial git info#319
albertvaka merged 1 commit into
masterfrom
albertvaka/do-not-drop-partial-git-info

Conversation

@albertvaka

Copy link
Copy Markdown
Collaborator

What does this PR do?

Removes the check that would not include git metadata if it was incomplete.

Motivation

We have relaxed the requirements on complete git info from the backend.

Review checklist (to be filled by reviewers)

  • Feature or bug fix MUST have appropriate tests (unit, integration, etc...)
  • PR title must be written as a CHANGELOG entry (see why)
  • Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
  • PR must have one changelog/ label attached. If applicable it should have the backward-incompatible label attached.
  • PR should not have do-not-merge/ label attached.
  • If Applicable, issue must have kind/ and severity/ labels attached at least.

@albertvaka
albertvaka force-pushed the albertvaka/do-not-drop-partial-git-info branch from 9ff2940 to 00bdc6c Compare November 30, 2022 14:14
@albertvaka
albertvaka merged commit 989037c into master Nov 30, 2022
@albertvaka
albertvaka deleted the albertvaka/do-not-drop-partial-git-info branch November 30, 2022 15:22
@albertvaka albertvaka added changelog/no-changelog Changes don't appear in changelog changelog/Added Added features results into a minor version bump and removed changelog/no-changelog Changes don't appear in changelog labels Dec 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/Added Added features results into a minor version bump

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants