Skip to content

ci: add notifications about new issues#3537

Merged
sobolevn merged 4 commits intowemake-services:masterfrom
Sehat1137:master
Oct 6, 2025
Merged

ci: add notifications about new issues#3537
sobolevn merged 4 commits intowemake-services:masterfrom
Sehat1137:master

Conversation

@Sehat1137
Copy link
Copy Markdown
Contributor

@Sehat1137 Sehat1137 commented Oct 6, 2025

I have made things!

Added notifications about new Issues to workflow

Checklist

  • I have double checked that there are no unrelated changes in this pull request (old patches, accidental config files, etc)
  • I have created at least one test case for the changes I have made
  • I have updated the documentation for the changes I have made
  • I have added my changes to the CHANGELOG.md

Comment thread .github/workflows/new-event-notify.yml Outdated
uses: sehat1137/telegram-notifier@5d130037eb80a33251f3afe8bc56447236acab77 # v 1.3.0
with:
tg-bot-token: ${{ secrets.TELEGRAM_BOT_TOKEN }}
tg-chat-id: ${{ vars.TELEGRAM_CHAT_ID }}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

why not secrets.TELEGRAM_CHAT_ID?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It`s not important, chat id isn't sensitive information. Do you prefer using secrets for this?

Comment thread .github/workflows/new-event-notify.yml Outdated
on:
issues:
types: [opened, labeled, reopened]
pull_request_target:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't think that I need PRs, only issues. Or is it some kind of a requirement?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

No, it's not nessesary, I will remove it

Comment thread .github/workflows/new-event-notify.yml Outdated
Comment thread .github/workflows/new-event-notify.yml Outdated
@Sehat1137 Sehat1137 changed the title ci: add notifications about new issues and pr ci: add notifications about new issues Oct 6, 2025
Copy link
Copy Markdown
Member

@sobolevn sobolevn left a comment

Choose a reason for hiding this comment

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

Let's try this for now, but I don't quite like that GITHUB_TOKEN is shared

@sobolevn sobolevn merged commit eceb0fe into wemake-services:master Oct 6, 2025
2 checks passed
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