Skip to content

Feat: AskPESU Integration - /ask command#73

Merged
sach-12 merged 2 commits intodevfrom
Josh9090/Integrated-askPESU
Sep 15, 2025
Merged

Feat: AskPESU Integration - /ask command#73
sach-12 merged 2 commits intodevfrom
Josh9090/Integrated-askPESU

Conversation

@joshua-rajj
Copy link
Contributor

@joshua-rajj joshua-rajj commented Sep 8, 2025

📌 Description

Summarize what this PR does:

added a /ask command that helps the user interact with askPESU

ℹ️ Fixes / Related Issues
#72


🧱 Type of Change

  • 🐛 Bug fix – Non-breaking fix for a functional/logic error
  • ✨ New feature – Adds functionality without breaking existing commands or APIs
  • ⚠️ Breaking change – Backward-incompatible change (commands, bot behavior, etc.)
  • 📝 Documentation update – README, comments, help text, etc.
  • 🧪 Test suite change – Adds/updates unit, integration, or manual tests
  • ⚙️ CI/CD pipeline update – GitHub Actions, Docker, pre-commit, etc.
  • 🧹 Refactor – Code cleanup, improvements, or style changes
  • 🐢 Performance improvement – Faster command responses or reduced resource use
  • 🕵️ Logging/debugging – Improved diagnostics, logs, or debug output
  • 🔧 Tooling – Scripts, benchmarks, or local dev improvements
  • 🔒 Security fix – Permissions, validation, or Discord API-related vulnerabilities
  • 🧰 Dependency update – Library or package updates

🧪 How Has This Been Tested?

  • Unit Tests (tests/unit/)
  • Integration Tests (tests/integration/)
  • Manual testing in Discord
  • CI / pre-commit run

Test Environment:

  • OS: (e.g., Linux)
  • Python: (e.g., 3.12)
  • Discord.py version: (e.g., 2.3.2)
  • Docker build tested

✅ Checklist

  • My code follows repo CONTRIBUTING.md guidelines
  • Self-review completed
  • Added/updated comments and docstrings
  • Updated relevant docs (README, help text, etc.)
  • No new warnings or errors introduced
  • Added/updated tests
  • All tests pass locally
  • Ran linting and formatting (pre-commit run --all-files)
  • Docker image builds and runs
  • Did not change config.json (unless with explicit permission)
  • Changes are backwards compatible (if applicable)
  • Feature flags or .env vars updated (if applicable)
  • Tested in multiple environments (if applicable)

🛠️ Affected Bot Areas

  • Commands (e.g., /ping, /help)
  • Event handling (on_message, on_ready, etc.)
  • Permissions/roles
  • Database/Storage
  • Discord API/Integrations
  • Bot startup/shutdown
  • CI / pre-commit
  • Dockerfile / build
  • Dependencies (requirements.txt)
  • Tooling/scripts
  • Other (specify in Additional Notes)

📸 Screenshots / Demos (if applicable)


🧠 Additional Notes

@joshua-rajj joshua-rajj requested review from a team and sach-12 as code owners September 8, 2025 19:17
@joshua-rajj joshua-rajj force-pushed the Josh9090/Integrated-askPESU branch from 7c4289f to 86e04a1 Compare September 9, 2025 10:20
@achyu-dev achyu-dev changed the title Josh9090/integrated ask pesu Feat: AskPESU Integration - /ask command Sep 9, 2025
{
"GUILD": {
"ID": 742797665301168220,
"ID": 1403764849598795786,
Copy link
Member

Choose a reason for hiding this comment

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

this shouldn't have been done, but i will let this go because i'm making major changes wrt config

Copy link
Member

@sach-12 sach-12 left a comment

Choose a reason for hiding this comment

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

LGTM

@sach-12 sach-12 enabled auto-merge September 15, 2025 17:47
@sach-12 sach-12 disabled auto-merge September 15, 2025 17:47
@sach-12 sach-12 merged commit 6a74872 into dev Sep 15, 2025
1 check passed
@sach-12 sach-12 linked an issue Sep 15, 2025 that may be closed by this pull request
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.

Integrating askPESU into discord bot

3 participants