Skip to content

chore: collapse welcome message content by default#882

Merged
Aaron ("AJ") Steers (aaronsteers) merged 2 commits intomainfrom
devin/1769123967-collapse-welcome-messages
Jan 22, 2026
Merged

chore: collapse welcome message content by default#882
Aaron ("AJ") Steers (aaronsteers) merged 2 commits intomainfrom
devin/1769123967-collapse-welcome-messages

Conversation

@aaronsteers
Copy link
Copy Markdown
Member

@aaronsteers Aaron ("AJ") Steers (aaronsteers) commented Jan 22, 2026

Summary

Restructures PR welcome messages to use collapsible HTML <details> sections, making the initial comment more compact. This follows the same pattern established in airbytehq/airbyte#72279.

Changes to both .github/pr-welcome-community.md and .github/pr-welcome-internal.md:

  • Content is now organized into two collapsible sections:
    • 💡 Show Tips and Tricks - Testing instructions and PR slash commands
    • 📚 Show Repo Guidance - Helpful resources, community contact info, and edit link
  • "If you have any questions" block moved inside the Repo Guidance section (at the end)
  • Added "Edit this welcome message" link to both files

Preview Links

Review & Testing Checklist for Human

  • Verify the <details> tags render correctly in GitHub PR comments (can test by creating a test PR)
  • Confirm template variables {{ .repo_name }} and {{ .branch_name }} still work inside collapsed sections

Notes

This is part of a broader effort to update welcome messages across all Airbyte repositories.

Link to Devin run: https://app.devin.ai/sessions/e4b88f0ca94c4ddfad32892a2be58fe6
Requested by: AJ Steers (Aaron ("AJ") Steers (@aaronsteers))


Open with Devin

Co-Authored-By: AJ Steers <aj@airbyte.io>
@devin-ai-integration
Copy link
Copy Markdown
Contributor

Original prompt from AJ Steers
Received message in Slack channel #ask-devin-ai:

@Devin please update the welcome message text (PR welcome and/or issue welcome) in all of our repos so that content is collapsed by default. Start with the main airbyte repo and then make a list of the rest you'll update after we merge the first
Thread URL: https://airbytehq-team.slack.com/archives/C08BHPUMEPJ/p1769113647989779

@devin-ai-integration
Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@github-actions
Copy link
Copy Markdown

👋 Greetings, Airbyte Team Member!

Here are some helpful tips and reminders for your convenience.

💡 Show Tips and Tricks

Testing This CDK Version

You can test this version of the CDK using the following:

# Run the CLI from this branch:
uvx 'git+https://github.com/airbytehq/airbyte-python-cdk.git@devin/1769123967-collapse-welcome-messages#egg=airbyte-python-cdk[dev]' --help

# Update a connector to use the CDK from this branch ref:
cd airbyte-integrations/connectors/source-example
poe use-cdk-branch devin/1769123967-collapse-welcome-messages

PR Slash Commands

Airbyte Maintainers can execute the following slash commands on your PR:

  • /autofix - Fixes most formatting and linting issues
  • /poetry-lock - Updates poetry.lock file
  • /test - Runs connector tests with the updated CDK
  • /prerelease - Triggers a prerelease publish with default arguments
  • /poe build - Regenerate git-committed build artifacts, such as the pydantic models which are generated from the manifest JSON schema in YAML.
  • /poe <command> - Runs any poe command in the CDK environment
📚 Show Repo Guidance

Helpful Resources

📝 Edit this welcome message.

Co-Authored-By: AJ Steers <aj@airbyte.io>
@github-actions
Copy link
Copy Markdown

PyTest Results (Fast)

3 826 tests  ±0   3 814 ✅ ±0   6m 30s ⏱️ -1s
    1 suites ±0      12 💤 ±0 
    1 files   ±0       0 ❌ ±0 

Results for commit 8db83d1. ± Comparison against base commit 4be34ee.

@aaronsteers Aaron ("AJ") Steers (aaronsteers) marked this pull request as ready for review January 22, 2026 23:29
Copilot AI review requested due to automatic review settings January 22, 2026 23:29
@aaronsteers Aaron ("AJ") Steers (aaronsteers) merged commit 9b23860 into main Jan 22, 2026
23 checks passed
@aaronsteers Aaron ("AJ") Steers (aaronsteers) deleted the devin/1769123967-collapse-welcome-messages branch January 22, 2026 23:29
Copy link
Copy Markdown
Contributor

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 restructures the PR welcome messages to use collapsible HTML sections, making the initial comment more compact and easier to navigate. The changes follow the pattern established in a related Airbyte repository PR.

Changes:

  • Wrapped content in collapsible <details> sections for both community and internal welcome messages
  • Reorganized content into "Tips and Tricks" and "Repo Guidance" sections
  • Added "Edit this welcome message" links to both templates

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/pr-welcome-internal.md Restructured internal welcome message with collapsible sections and added edit link
.github/pr-welcome-community.md Restructured community welcome message with collapsible sections and added edit link

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional flags.

Open in Devin Review

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