Skip to content
This repository has been archived by the owner on Nov 11, 2020. It is now read-only.

chore: update cert-manager #50

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gazal-k
Copy link

@gazal-k gazal-k commented Apr 1, 2020

v0.11.0 does not have apiVersion in Chart.yaml. So PR builds that lint helm charts in env repos fail with: [ERROR] Chart.yaml: apiVersion is required. The value must be either "v1" or "v2"

v0.11.0 does not have apiVersion in Chart.yaml. So PR builds that lint helm charts in env repos fail with: `[ERROR] Chart.yaml: apiVersion is required. The value must be either "v1" or "v2"`
@jstrachan
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign pmuir
You can assign the PR to them by writing /assign @pmuir in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@gazal-k
Copy link
Author

gazal-k commented Apr 1, 2020

I assume there is nothing preventing us from upgrading. Not going all the way to the latest version though

@gazal-k
Copy link
Author

gazal-k commented Apr 7, 2020

I assume there is nothing preventing us from upgrading. Not going all the way to the latest version though

fair warning; we haven't tested this yet

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

Successfully merging this pull request may close these issues.

2 participants