-
Notifications
You must be signed in to change notification settings - Fork 5
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
fix(apps/prod/jenkins-beta): remove useless post step #659
fix(apps/prod/jenkins-beta): remove useless post step #659
Conversation
Pull Request ReviewSummaryThis pull request adds some ns config to the Jenkins Beta application. Changes Made
Potential ProblemsNo potential problems found. Fixing SuggestionsNo fixing suggestions needed. |
It is difficult to ascertain if there are any potential problems with the changes made in this pull request as there is no context provided. However, it is possible that deleting files from the A possible suggestion for fixing this pull request is to provide more context on why these files were deleted and what impact this will have on the |
Also you should delete the post.yaml and update the kustomzation file @purelind |
No potential problems have been identified in this pull request. Suggestion: The pull request looks good as it is, no suggestions are needed. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: wuhuizuo The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
[LGTM Timeline notifier]Timeline:
|
remove useless post.