Skip to content

deploy.sh and deploy.bat duplicate deployment logic across platforms - #718

Merged
JSE19 merged 1 commit into
JSE-ORG:mainfrom
LawalRahman:main
Jul 29, 2026
Merged

deploy.sh and deploy.bat duplicate deployment logic across platforms#718
JSE19 merged 1 commit into
JSE-ORG:mainfrom
LawalRahman:main

Conversation

@LawalRahman

Copy link
Copy Markdown
Contributor

Pull Request Checklist

Thank you for contributing! Please review the following checklist to ensure your PR is ready for review.

Summary

  • Provide a clear and concise description of the changes.
  • Link to any related issues using Fixes #issue or Related to #issue.

Testing

  • All new code is covered by unit tests where applicable.
  • Existing tests pass locally (cargo test).
  • Added tests for edge cases and error conditions.
  • Updated any integration tests if necessary.

Documentation

  • Updated README.md if changes affect users.
  • Updated docstrings/comments for new/modified functions.
  • Added examples or updated existing examples if applicable.

Code Quality

  • Follows the project's coding style and conventions.
  • No commented-out code or debug statements left in the codebase.
  • Variables and functions are named descriptively.
  • Code is properly formatted (cargo fmt).
  • No new clippy warnings (cargo clippy).

Breaking Changes

  • If this PR introduces breaking changes, describe them and provide migration steps.
  • Updated version in Cargo.toml if appropriate (following semver).

Additional Notes

  • Any other relevant information for reviewers.

Please ensure all checkboxes are checked before requesting a review.

Closes #705
Closes #681
Closes #680
Closes #679

@drips-wave

drips-wave Bot commented Jul 29, 2026

Copy link
Copy Markdown

@LawalRahman Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@JSE19
JSE19 merged commit 68b4914 into JSE-ORG:main Jul 29, 2026
3 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants