Skip to content

docs: delete duplicate root CONTRIBUTING.md, add release-drafter ref to RELEASES.md#917

Draft
Aaron ("AJ") Steers (aaronsteers) wants to merge 3 commits intomainfrom
devin/1771619414-fix-contributing-docs
Draft

docs: delete duplicate root CONTRIBUTING.md, add release-drafter ref to RELEASES.md#917
Aaron ("AJ") Steers (aaronsteers) wants to merge 3 commits intomainfrom
devin/1771619414-fix-contributing-docs

Conversation

@aaronsteers
Copy link
Contributor

@aaronsteers Aaron ("AJ") Steers (aaronsteers) commented Feb 20, 2026

docs: delete duplicate root CONTRIBUTING.md, add release-drafter ref to RELEASES.md

Summary

Follow-up to #916, which added a root-level CONTRIBUTING.md with a Releasing section but missed the existing detailed contributing docs at docs/CONTRIBUTING.md and docs/RELEASES.md.

This PR:

  • Deletes the duplicate root CONTRIBUTING.md (the canonical guide already lives at docs/CONTRIBUTING.md and is linked from README.md line 30)
  • Adds a semantic-pr-release-drafter reference to the top of docs/RELEASES.md, which is the dedicated releases page for this repo

Updates since last revision

Per reviewer feedback:

  • Root CONTRIBUTING.md is now deleted instead of kept as a redirect (it was a duplicate that shouldn't have been created)
  • Reverted changes to docs/CONTRIBUTING.md — its Release Management section already just links to RELEASES.md, which is the right place for release content
  • Moved the semantic-pr-release-drafter reference into docs/RELEASES.md instead

Review & Testing Checklist for Human

  • Confirm README.md line 30 still links to docs/CONTRIBUTING.md correctly (no broken contributor onboarding path after root file deletion)
  • Verify the new semantic-pr-release-drafter blurb at the top of docs/RELEASES.md reads well alongside the existing publishing instructions below it

Notes

Requested by: Aaron ("AJ") Steers (@aaronsteers)
Devin session

…release-drafter references

Co-Authored-By: AJ Steers <aj@airbyte.io>
@devin-ai-integration
Copy link
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

👋 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/1771619414-fix-contributing-docs#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/1771619414-fix-contributing-docs

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.

CONTRIBUTING.md Outdated
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ditto. Delete this dupe file.

…d as canonical

Co-Authored-By: AJ Steers <aj@airbyte.io>
Comment on lines 98 to 100
This project uses [`semantic-pr-release-drafter`](https://github.com/aaronsteers/semantic-pr-release-drafter) for automated release management. For more information, see the [Releasing Guide](https://github.com/aaronsteers/semantic-pr-release-drafter/blob/main/docs/releasing.md).

Please also see the [Release Management](./RELEASES.md) guide for repo-specific information on how to perform releases and pre-releases.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Don't put this content here, since we have a dedicated page already for releases. Put the content into RELEASES.md - or if there's no need for that page anymore (because our new docs cover everything) delete it, in which case the content can live here.

Copy link
Contributor

Choose a reason for hiding this comment

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

Done — reverted docs/CONTRIBUTING.md back to its original text (just links to RELEASES.md), deleted the duplicate root CONTRIBUTING.md, and moved the semantic-pr-release-drafter reference into docs/RELEASES.md at the top of the page. The existing repo-specific content in RELEASES.md is preserved.


Devin session

… to RELEASES.md

Co-Authored-By: AJ Steers <aj@airbyte.io>
@devin-ai-integration devin-ai-integration bot changed the title docs: link root CONTRIBUTING.md to existing docs and add releasing references docs: delete duplicate root CONTRIBUTING.md, add release-drafter ref to RELEASES.md Feb 20, 2026
@github-actions
Copy link

PyTest Results (Fast)

3 869 tests  ±0   3 857 ✅ ±0   6m 38s ⏱️ -13s
    1 suites ±0      12 💤 ±0 
    1 files   ±0       0 ❌ ±0 

Results for commit 2a1b63e. ± Comparison against base commit 7f41401.

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.

1 participant