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

Add electrical, safety and instrumentation as prioritized sectors #262

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Mawik05
Copy link

@Mawik05 Mawik05 commented Feb 13, 2025

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

Add electrical, safety and instrumentation equipment as prioritized sectors.

How can you best review this? 🧐

Run some builds and check that build time doesn't increase and that there are not too many prioritized sectors.

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.

1 participant