Skip to content

Conversation

AndreasArvidsson
Copy link
Member

@AndreasArvidsson AndreasArvidsson commented Jan 6, 2025

We should not show notifications without telling the user it comes from Cursorless

Checklist

  • [/] I have added tests
  • [/] I have updated the docs and cheatsheet
  • [/] I have not broken the cheatsheet

@AndreasArvidsson AndreasArvidsson requested a review from a team as a code owner January 6, 2025 16:38
@pokey pokey added this pull request to the merge queue Jan 7, 2025
Merged via the queue into main with commit c2d3766 Jan 7, 2025
15 checks passed
@pokey pokey deleted the communityRequirements branch January 7, 2025 14:56
cursorless-bot pushed a commit that referenced this pull request Jan 7, 2025
We should not show notifications without telling the user it comes from
Cursorless

## Checklist

- [/] I have added
[tests](https://www.cursorless.org/docs/contributing/test-case-recorder/)
- [/] I have updated the
[docs](https://github.com/cursorless-dev/cursorless/tree/main/docs) and
[cheatsheet](https://github.com/cursorless-dev/cursorless/tree/main/cursorless-talon/src/cheatsheet)
- [/] I have not broken the cheatsheet
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.

2 participants