Skip to content

Commit

Permalink
Drop @here in fjord threads (#267)
Browse files Browse the repository at this point in the history
@Heres are not generally useful in threads due to inclusion rules.
  • Loading branch information
Shadowfiend authored Jun 19, 2023
2 parents 72a3562 + 16b6e6d commit d5290ef
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 6 deletions.
4 changes: 3 additions & 1 deletion .github/workflows/build-and-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,9 @@ jobs:
- name: Output environment
run: |
echo '${{ github.ref }}'
echo '${{ toJSON(github.event) }}'
echo <<-EOF
${{ toJSON(github.event) }}
EOF
outputs:
# Run builds for all non-comment triggers.
should-build: >-
Expand Down
3 changes: 0 additions & 3 deletions discord-scripts/discord-webhook.ts
Original file line number Diff line number Diff line change
Expand Up @@ -36,15 +36,12 @@ export default async function webhookDiscord(
)

if (existingThread) {
await existingThread.send("@here")
await existingThread.send(message)
} else {
const newThread = await channel.threads.create({
name: title,
autoArchiveDuration: 60,
reason: message,
})
await newThread.send("@here")
if (tagUser !== "0") {
await Promise.all(
memberIds.map((id) => newThread.members.add(id.trim())),
Expand Down
2 changes: 0 additions & 2 deletions discord-scripts/fjord.ts
Original file line number Diff line number Diff line change
Expand Up @@ -194,7 +194,6 @@ export default async function manageFjord(discordClient: Client, robot: Robot) {
name: `Stale issues: ${repositoryOwner} ${repositoryName}`,
autoArchiveDuration: 60,
})
await thread.send("@here")
await thread.send(response.data)
}
})
Expand Down Expand Up @@ -249,7 +248,6 @@ export default async function manageFjord(discordClient: Client, robot: Robot) {
name: `Issues: ${repositoryOwner} ${repositoryName}`,
autoArchiveDuration: 60,
})
await thread.send("@here")
await thread.send(response.data)
}
})
Expand Down

0 comments on commit d5290ef

Please sign in to comment.