farmer icon indicating copy to clipboard operation
farmer copied to clipboard

Sorting resources in Farmer/Arm for ease of maintenance.

Open ninjarobot opened this issue 1 year ago • 3 comments

The changes in this PR are as follows:

  • Sorts modules for ARM resources in alphabetical order.

I have read the contributing guidelines and have completed the following:

  • [x] Tested my code end-to-end against a live Azure subscription.
  • [x] Updated the documentation in the docs folder for the affected changes.
  • [x] Written unit tests against the modified code that I have made.
  • [x] Updated the release notes with a new entry for this PR.
  • [x] Checked the coding standards outlined in the contributions guide and ensured my code adheres to them.

ninjarobot avatar Mar 26 '24 20:03 ninjarobot

@isaacabraham the project has gotten pretty big and needs a little more organization. I made a few changes here to reduce the dependencies between IArmResource modules and allow most of them to be alphabetically sorted. What do you think? Any recommendations to make it easier to navigate?

ninjarobot avatar Mar 30 '24 11:03 ninjarobot

@ninjarobot This looks good - assuming that the conflicts can be resolved + Matt's questions addressed?

isaacabraham avatar Jul 03 '24 11:07 isaacabraham

@ninjarobot This looks good - assuming that the conflicts can be resolved + Matt's questions addressed?

Yes, they can be, I will need to get back to this cleanup.

ninjarobot avatar Jul 03 '24 14:07 ninjarobot