Skip to content

Conversation

@shaljam
Copy link
Contributor

@shaljam shaljam commented Apr 8, 2025

Submit a pull request

CLA

  • I have signed the Stream CLA (required).
  • The code changes follow best practices
  • Code changes are tested (add some information if not applicable)

Description of the pull request

totalimmersion
totalimmersion previously approved these changes Apr 8, 2025

def delete(self) -> StreamResponse:
return self.client.delete(self.url)
def delete(self, hard: bool = False) -> StreamResponse:
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this a breaking change, or you can call delete without passing the hard flag?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is not a breaking change. You can. It has a default.

slavabobik
slavabobik previously approved these changes Apr 9, 2025
@shaljam shaljam dismissed stale reviews from slavabobik and totalimmersion via 7836036 April 9, 2025 09:57
@shaljam shaljam merged commit 564e947 into master Apr 9, 2025
6 checks passed
@shaljam shaljam deleted the fix-tests branch April 9, 2025 15:23
@github-actions github-actions bot mentioned this pull request Jun 18, 2025
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.

4 participants