Skip to content
Discussion options

You must be logged in to vote

Hi! Thanks for sharing this — totally agree with your points.

From our side, we’re seeing something similar. The current approach does the job, but there are definitely areas where things could be clearer, especially around edge cases. In some scenarios, it’s not entirely obvious what the expected behavior should be, which can lead to confusion when trying to debug or extend the logic.

Improving the documentation would go a long way here, particularly by outlining those edge cases more explicitly. That would make it easier for anyone jumping into the codebase to understand the intended flow without needing to reverse-engineer it.

Also +1 on simplifying parts of the implementation. There a…

Replies: 3 comments

Comment options

You must be logged in to vote
0 replies
Comment options

You must be logged in to vote
0 replies
Comment options

You must be logged in to vote
0 replies
Answer selected by jhojan16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug GitHub or a GitHub feature is not working as intended Apps API and Webhooks Discussions related to GitHub's APIs or Webhooks Welcome 🎉 Used to greet and highlight first-time discussion participants. Welcome to the community! source:ui Discussions created via Community GitHub templates
2 participants