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

feat: Repository migration #2

Merged
merged 10 commits into from
Aug 23, 2022
Merged

feat: Repository migration #2

merged 10 commits into from
Aug 23, 2022

Conversation

Raffy23
Copy link
Member

@Raffy23 Raffy23 commented Aug 16, 2022

This PR

  • Adds a LICENSE file, keptn-contrib GitHub automation workflows (Release, Integration tests ...)
  • Adds renovate, reviewdog configuration files
  • Includes a compatibility matrix in the README.md file

@github-actions
Copy link
Contributor

github-actions bot commented Aug 16, 2022

Unit Test Results

67 tests   67 ✔️  0s ⏱️
13 suites    0 💤
  1 files      0

Results for commit 0d6c3a2.

♻️ This comment has been updated with latest results.

Signed-off-by: Raphael Ludwig <[email protected]>
@Raffy23 Raffy23 force-pushed the feat/repository-migration branch from a1f43db to 97eee7d Compare August 17, 2022 06:44
@Raffy23 Raffy23 marked this pull request as ready for review August 17, 2022 15:21
Copy link
Contributor

@christian-kreuzberger-dtx christian-kreuzberger-dtx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left a comment.
Also I saw that chart/charts/common-0.1.0.tgz is in the PR... not sure if that is on purpose?

.github/workflows/CI.yml Outdated Show resolved Hide resolved
installer/manifests/common/Chart.yaml Outdated Show resolved Hide resolved
Signed-off-by: Raphael Ludwig <[email protected]>
@Raffy23
Copy link
Member Author

Raffy23 commented Aug 22, 2022

Left a comment. Also I saw that chart/charts/common-0.1.0.tgz is in the PR... not sure if that is on purpose?

It was like this in the original repository, I removed the common chart and used the similar helpers that we have in the job-executor-service Chart.

Raphael Ludwig added 2 commits August 22, 2022 10:08
Signed-off-by: Raphael Ludwig <[email protected]>
@Raffy23 Raffy23 force-pushed the feat/repository-migration branch from 8f2f269 to 0d6c3a2 Compare August 23, 2022 14:30
Copy link
Contributor

@christian-kreuzberger-dtx christian-kreuzberger-dtx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions
Copy link
Contributor

The following Docker Images have been built:
ghcr.io/keptn-contrib/helm-service:0.0.2-dev-PR-2,ghcr.io/keptn-contrib/helm-service:0.0.2-dev-PR-2.202208231430

@Raffy23 Raffy23 merged commit 5c7abf9 into main Aug 23, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants