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

Add missing switches for Grafana manifests. #3161

Closed
wants to merge 1 commit into from
Closed

Add missing switches for Grafana manifests. #3161

wants to merge 1 commit into from

Conversation

yigitemres
Copy link
Contributor

@yigitemres yigitemres commented Aug 20, 2020

  • Have you signed the Contributor License Agreement?
  • Have you checked that there aren't other open pull requests for the same manifest update/change?
  • Have you validated your manifest locally with winget validate <manifest>, where <manifest> is the name of the manifest you're submitting?
  • Have you tested your manifest locally with winget install -m <manifest>?

Microsoft Reviewers: Open in CodeFlow

@wingetbot
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ghost
Copy link

ghost commented Aug 20, 2020

yigitemres The package manager bot determined there was an issue with the pull request. Make sure the ID is of the form publisher.appname and that the folder structure is publisher\appname.
For details on the error, see the details link below in the build pipeline.

@yigitemres
Copy link
Contributor Author

@KevinLaMS Do I have to fix manfiests one by one or any MS reviewer can review it and force to merge? I will be happy if you answer that.

@yigitemres
Copy link
Contributor Author

Closing pr in order to do fixes one by one: #125

@yigitemres yigitemres closed this Aug 20, 2020
@yigitemres yigitemres deleted the grafana-switchfix branch August 20, 2020 23:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants