Skip to content

Conversation

@Strepto
Copy link
Collaborator

@Strepto Strepto commented Oct 20, 2025

What have I made? πŸ‘©β€πŸ’»

This adds priority sectors for MECH. This MAY be costly, so lets see how many sectors this produces on large assets.

How can you best review this? 🧐

Read and critique the selection methods I have picked for MECH.

Did I remember everything checklist?

  • I made some awesome changes! 😎
  • I left the code in a better state than when I started working on it ✨
  • I wrote unit and/or integration tests πŸ‘Ύ
  • I have considered any security implications, and requested review of them explicitly πŸ”
  • I verified that it checks the acceptance criteria. πŸ“œ
  • I have tested this change in the Echo DEV environment, or requested someone to test it for me πŸš€
  • I updated the documentation/readme πŸ“š
  • I made the PR title descriptive and human-readable πŸ‘¨β€πŸ‘©β€πŸ‘¦β€πŸ‘¦
  • Existing unreported issues I found but could not fix was added as a new Issues 🚧

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants