Update Rule “azure-budgets/rule”

[AttackOnMorty]

1. What triggered this change? (PBI link, Email Subject, conversation + reason, etc)

Problem found by myself

2. What was changed?

Updated invalid link

3. Did you do pair or mob programming (and list names)?

No

[JackDevAU]

Hey @tiagov8,

This seems to be a false positive. Its picking up an issue with the list of numbered items. But in this case it makes sense.

I'm unsure why it ran and didn't comment though, I'll have a look into that today.

This PR can be merged fine.

[adamcogan]

Hi @JackDevAU
⚠️ I much prefer you fix it first.
I think that's a better approach, so we know it's properly fixed
-a

[JackDevAU]

The Issue:

1. Please approve

\< As per https://ssw.com.au/rules/azure-budgets \>
:::
:::

3. If the request is approved, remember to add a cost-category tag to the new resource once it is created

The issue is being picked up due to the above ordered list being out of sync. Since it doesn't know that the 1 is part of an email and not a continuation of the 2 further up.

[JackDevAU]

To fix the issue we can change the ordered list from 1,2,3 etc... to 1. 1. 1.

[JackDevAU]

Update

As per my conversation with @bradystroud, due to how forked PR's work - we can't currently comment on PR's (like this one). So for now - I will disable the validator for forked branches until we have a workaround as seen in @wicksipedia video on this subject. https://youtu.be/h05ZuwgOa8Q?si=UWqinPh6OKcYlJ7F

[wicksipedia]

I will disable the validator for forked branches until we have a workaround as seen in @wicksipedia video on this subject. https://youtu.be/h05ZuwgOa8Q?si=UWqinPh6OKcYlJ7F

remember to smash the like and subscribe button

[github-actions]

Howzit @AttackOnMorty,

This PR has been here a while.

Did you know you should avoid merge debt?

1. Please action (e.g. get a review) and merge or close

Thanks!