[go: up one dir, main page]

fix!: Sendmail shouldn't implicitly commit transaction by ankush [frappe] PR#35301

From: https://github.com/frappe/frappe/pull/35301
Date: 2025-12-17 16:12:34+05:30

  • test: commit to trigger sendmail
  • chore: remove unncessary print
  • fix!: sendmail now=True after current transaction is commited



Diagnostics

pre-commit failed for source commit: 9979314b327c3223af48b3676c684c19cc8fc594
warning: The following rules have been removed and ignoring them has no effect:
    - UP038

1 file reformatted, 3 files left unchanged




Suspicious changes found (Dokos Only / dokos):
* dde30fe1e35ad1f752fd0d03c6b91d75fcf1e32c test: commit to trigger sendmail
```diff

```



Checkout instructions
# Checkout locally
git fetch upstream
git switch ft-pr-35301

# Alternatively, re-take the changes
git switch develop
ft take ft-pr-35301

# Make changes then rebase
git rebase -i develop

# Fix or ignore conflicts
git checkout --theirs .
git rebase --continue

# Force-push changes
git push --force-with-lease

Merge request reports

Loading