Skip to content

fix integration test failure slack message#517

Open
karawoo wants to merge 1 commit intomainfrom
fix-message
Open

fix integration test failure slack message#517
karawoo wants to merge 1 commit intomainfrom
fix-message

Conversation

@karawoo
Copy link
Collaborator

@karawoo karawoo commented Mar 9, 2026

The top-level text at line 73 is apparently only a fallback. I was expecting it to appear in the message but it doesn't; this PR adds the appropriate text in the body of the message.

Checklist

  • Does this change update NEWS.md (referencing the connected issue if necessary)?
  • Does this change need documentation? Have you run devtools::document()?
  • Does this change resolve integration test failures for a preview build of Connect? If so (and if you are on the Connect team), please be sure to update the Connect changelog to reflect that users will need to upgrade their version of connectapi.

Copy link
Collaborator

@jonkeane jonkeane left a comment

Choose a reason for hiding this comment

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

Thanks for this!

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