Skip to content

chore: update bot messages #4517

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Apr 16, 2025
Merged

Conversation

moskalakamil
Copy link
Collaborator

Summary

Update bot message

Changes

  • updated roadmap link
  • added issue boost link
  • add missing func - hide previous comments on feat req

Test plan

You can see workflow here: https://github.com/moskalakamil/react-native-video/issues?q=sort:updated-desc+is:issue+is:open

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the bot messages for issue tracking, including updating the roadmap link, adding an Issue Boost link, and introducing a function call to hide previous comments.

  • Updates the FEATURE_REQUEST, BUG_REPORT, MISSING_INFO, and OUTDATED_VERSION messages with new content and formatting
  • Introduces the hidePreviousComments function call to avoid duplicate messages
Comments suppressed due to low confidence (2)

.github/scripts/validate.js:192

  • Consider adding unit tests to verify that hidePreviousComments correctly hides previous bot comments in various scenarios.
await hidePreviousComments({github, context});

.github/scripts/validate.js:37

  • [nitpick] Review the formatting of the multiline FEATURE_REQUEST message template to ensure that extra indentation within the template literal does not lead to unintended whitespace in the output.
FEATURE_REQUEST: `Thanks for the feature request! 🚀 ...`

@KrzysztofMoch KrzysztofMoch merged commit 2bdc0da into TheWidlarzGroup:master Apr 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

2 participants