Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make ResourceStack fully injectable #6591

Merged
merged 1 commit into from
Nov 24, 2022
Merged

Conversation

Nokel81
Copy link
Collaborator

@Nokel81 Nokel81 commented Nov 16, 2022

Signed-off-by: Sebastian Malton sebastian@malton.name

@Nokel81 Nokel81 added bug Something isn't working area/metrics All the things related to metrics blocker labels Nov 16, 2022
@Nokel81 Nokel81 added this to the 6.2.0 milestone Nov 16, 2022
@Nokel81 Nokel81 requested a review from a team as a code owner November 16, 2022 20:11
@Nokel81 Nokel81 requested review from Iku-turso and aleksfront and removed request for a team November 16, 2022 20:11
@Nokel81 Nokel81 added chore and removed bug Something isn't working area/metrics All the things related to metrics blocker labels Nov 17, 2022
@Nokel81 Nokel81 modified the milestones: 6.2.0, 6.3.0 Nov 17, 2022
@Nokel81 Nokel81 changed the title Fix not being able to install Lens Metrics Make ResourceStack fully injectable Nov 17, 2022
@github-actions
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

@github-actions
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

Signed-off-by: Sebastian Malton <sebastian@malton.name>
@github-actions
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

@@ -0,0 +1,19 @@
/**
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: name should be src/renderer/kubectl/kubectl-apply-all.injectable.ts since the main partner is src/main/kubectl/kubectl-apply-all.injectable.ts

@Nokel81 Nokel81 merged commit 6142aad into master Nov 24, 2022
@Nokel81 Nokel81 deleted the fix-metrics-apply-macOS branch November 24, 2022 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants