search

Azure-Samples / langfuse-on-azure

An easy way to deploy the Langfuse observability platform to Azure Container Apps with Entra authentication.
MIT License
37 stars 4 forks source link

Change permissions location to match docs #7

Closed pamelafox closed 7 months ago

pamelafox commented 7 months ago

Purpose

Just reorg permissions to exactly match the docs at https://docs.github.com/en/code-security/code-scanning/integrating-with-code-scanning/uploading-a-sarif-file-to-github Also removed env variable so that it doesnt see the spurious errors. It seems to upload the issues fine without verbose mode needed.

Does this introduce a breaking change?

[ ] Yes
[X] No

Pull Request Type

What kind of change does this Pull Request introduce?

[X] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test