Skip to content

Conversation

JamieByrneHDRUK
Copy link
Contributor

Screenshots (if relevant)

Describe your changes

Issue ticket link

Environment / Configuration changes (if applicable)

Requires migrations being run?

If not using the pre-push hook. Confirm tests pass:

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have added appropriate unit tests
  • I have created mocks for unit tests (where appropriate)
  • I have added appropriate Behat tests to confirm AC (if applicable)
  • I have added Swagger annotations for new endpoints (if applicable)
  • I have added audit logs for new operation logic (if applicable)
  • I have added new environment variables to the .env.example file (if applicable)
  • I have added new environment variables to terraform repository (if applicable)

Copy link

🚨 Invalid PR title format!

Your PR title must follow one of these formats:

  • feat(GAT-1234): Your title
  • fix!(GAT-5678): Breaking change
  • RELEASE: vX.Y.Z

Please update your PR title accordingly. 😊

@JamieByrneHDRUK JamieByrneHDRUK changed the title fix():Sorin Cant Be Mad fix(GAT-6216):Sorin Cant Be Mad Feb 21, 2025
Copy link

🚨 Invalid PR title format!

Your PR title must follow one of these formats:

  • feat(GAT-1234): Your title
  • fix!(GAT-5678): Breaking change
  • RELEASE: vX.Y.Z

Please update your PR title accordingly. 😊

@JamieByrneHDRUK JamieByrneHDRUK changed the title fix(GAT-6216):Sorin Cant Be Mad fix(GAT-6216): Add in force and rebase to avoid conflicts Feb 21, 2025
Copy link

🎉 Great job! Your PR title follows the correct format. 🚀

@JamieByrneHDRUK JamieByrneHDRUK marked this pull request as ready for review February 21, 2025 14:39
@soringumeni1 soringumeni1 merged commit bdaca07 into dev Feb 21, 2025
2 of 4 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
Development

Successfully merging this pull request may close these issues.

3 participants