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

🪲 Bug Report - Policy Workflow CRLF to LF warning causing errors #348

Closed
4pplied opened this issue Oct 21, 2022 · 0 comments · Fixed by #350
Closed

🪲 Bug Report - Policy Workflow CRLF to LF warning causing errors #348

4pplied opened this issue Oct 21, 2022 · 0 comments · Fixed by #350
Assignees

Comments

@4pplied
Copy link
Contributor

4pplied commented Oct 21, 2022

Describe the bug

During the Commercial Cloud Update Policy Library it is returning an error of replacing CRLF with LF which is not an affecting formatting to the policies/workflow.

To Reproduce

Steps to reproduce the behaviour:

  1. Look in the Update Policy Library action history in Azure/ALZ-Bicep

Expected behaviour

Update the policy library and green tick the workflow

Screenshots 📷

image

Correlation ID

N/A

Additional context

As this workflow runs on Ubuntu and is a Unix OS, we should set git.autocrlf to input in order to instruct Git to deal with this accordingly. This is common amongst Windows / Unix checkouts and cross-platform developing. See documentation:

https://git-scm.com/book/en/v2/Customizing-Git-Git-Configuration#Formatting-and-Whitespace

image

@4pplied 4pplied added the bug label Oct 21, 2022
@ghost ghost added the Needs: Triage 🔍 Needs triaging by the team label Oct 21, 2022
@4pplied 4pplied self-assigned this Oct 21, 2022
@jtracey93 jtracey93 linked a pull request Oct 21, 2022 that will close this issue
10 tasks
@jtracey93 jtracey93 removed the Needs: Triage 🔍 Needs triaging by the team label Oct 21, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Nov 20, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants