grafana / grafana-kiosk

Kiosk Utility for Grafana
Apache License 2.0
362 stars 59 forks source link

AzureAD Auto-Login Local Override #107

Closed friedpope closed 10 months ago

friedpope commented 1 year ago

This is for Issue #106. I think it's ready to merge, the changes are pretty simple.

CLAassistant commented 1 year ago

CLA assistant check
All committers have signed the CLA.

friedpope commented 1 year ago

I'm not sure what that last codeclimate issue is on line 1, I'm happy to fix it if anyone can add some more detail as to what's wrong.

briangann commented 1 year ago

it's a "gofmt" issue, just formatting that can be fixed post-merge.

friedpope commented 1 year ago

are there docs to be added for this option?

Sure, I'll work on an update to README.md tomorrow.

briangann commented 1 year ago

can you run mage build:format to update the source files to pass the "gofmt" check? if not, i can run it post-merge, looks like this is ready to release!