Skip to content
This repository has been archived by the owner on Oct 23, 2024. It is now read-only.

chore: Sync main with beta #66

Open
wants to merge 14 commits into
base: beta
Choose a base branch
from
Open

chore: Sync main with beta #66

wants to merge 14 commits into from

Conversation

d2iq-mergebot
Copy link
Contributor

Sync latest main with beta branch

kaiwalyajoshi and others added 12 commits January 23, 2023 17:31
Add GHA to sync main and beta branches.
Remove use of deprecated downloads.mesosphere.io
Revert to previous bucket name as builds will fail with downloads.d2iq.com.
Add codeowners file.
Update workflow to always update the beta branch when pushes are made to main.
Bringing all CI builds for this repo from TC to GH Actions
Add the missing 'v' prefix to SemVer regex, without this we
default safely to main which isn't what we want for dev and
rc builds.
Release DKP-Insights v0.4.0
Release DKP-Insights v0.4.1

Co-authored-by: d2iq-mergebot <[email protected]>
fixing Sync GHA workflow.
Release DKP-Insights v0.3.2

Co-authored-by: d2iq-mergebot <[email protected]>
@kaiwalyajoshi
Copy link
Contributor

Precommit fails due to missing body in one of the commits, shouldn't be a reason to prevent merging this PR.

@MessKon
Copy link
Contributor

MessKon commented May 19, 2023

Precommit fails due to missing body in one of the commits, shouldn't be a reason to prevent merging this PR.

if we merge a commit which gets the pre-commit check to fail, won't all future pre-commit checks fail due to this commit?
Also, why didn't the original mergebot commit cover all the changes to be synced?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants