Skip to content

Add CONTRIBUTING.md and trim README#278

Draft
yahonda wants to merge 2 commits intorsim:masterfrom
yahonda:add-contributing-md
Draft

Add CONTRIBUTING.md and trim README#278
yahonda wants to merge 2 commits intorsim:masterfrom
yahonda:add-contributing-md

Conversation

@yahonda
Copy link
Copy Markdown
Collaborator

@yahonda yahonda commented Apr 30, 2026

Summary

  • Add CONTRIBUTING.md modelled on rsim/oracle-enhanced: how to report issues, submit PRs, develop in the devcontainer (added in Add devcontainer environment and CI #276), reproduce randomized spec runs by seed (with --bisect), and run RuboCop with the same commands as CI.
  • Move the README's TESTS section (manual schema setup, bundle install, rake spec) into a new "Manual setup (without devcontainer)" section in CONTRIBUTING.md, and replace the README section with a one-line pointer.
  • Replace the hand-maintained CONTRIBUTORS list in README with a thank-you note plus a link to the canonical list at https://github.com/rsim/ruby-plsql/graphs/contributors so it stops drifting in the repo.

Notes

Test plan

  • Verify the rendered CONTRIBUTING.md and README.md on GitHub read naturally.
  • Confirm the CONTRIBUTING.md link from README resolves once the PR is merged.

🤖 Generated with Claude Code

yahonda added 2 commits April 30, 2026 16:00
Document how to report issues, submit PRs, work in the devcontainer,
run the spec suite (including seed-based reproduction and bisect), and
run RuboCop with the same commands as CI. Modelled after rsim/oracle-enhanced
to keep contributor guidance consistent across the two adapters.
Migrate the manual database setup, dependency install, and test
invocation steps from README's TESTS section into CONTRIBUTING.md (under
a new "Manual setup (without devcontainer)" section) and replace the
README section with a pointer. The README's hand-maintained CONTRIBUTORS
list is replaced with a thank-you note plus a link to
github.com/rsim/ruby-plsql/graphs/contributors so the canonical list
lives on GitHub instead of drifting in the repo.
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