Skip to content

Conversation

vpnu
Copy link
Collaborator

@vpnu vpnu commented Jul 4, 2025

🚧 Suggest a change

  • Remove statuses mentioning crates & crate validation, no validation was actually implemented
  • Rename Pod to Agent to match docs language
  • Remove commented out code and unused repeated methods

📝 Pre-merge checklist

Ready to merge? Do not merge until all checks are satisfied.

  • 💹 Have all required CI checks passed on the most recent commit?
  • ✒️ Is the PR title a valid and meaningful conventional-commit message? ie. type(scope): summary
  • 💥 Are breaking changes declared in the PR title in conventional-commit style? ie. type!(scope): summary
  • 🎨 Does new code follow the code style of this project?
  • 🔍 Has new code been spellchecked and linted?
  • 📖 Have docs been updated where necessary?
  • 💩 Have commits been checked for accidental file inclusions?

@vpnu vpnu requested review from jaybeeelsdon, beforan and a team as code owners July 4, 2025 09:54
@vpnu vpnu changed the title fix:change status updates for All In One DemoMode fix: change status updates for All In One DemoMode Jul 4, 2025
Copy link

github-actions bot commented Jul 4, 2025

✒️ PR Title Commitlint - ✔️ Lint success!

Copy link
Collaborator

@MatthewWong68 MatthewWong68 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Code looks cleaner.

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.

3 participants