Skip to content

Recommit Rails 8.1 schema#831

Open
zetter-rpf wants to merge 1 commit into
mainfrom
fix-schema
Open

Recommit Rails 8.1 schema#831
zetter-rpf wants to merge 1 commit into
mainfrom
fix-schema

Conversation

@zetter-rpf
Copy link
Copy Markdown
Contributor

Rails 8 changed the schema and ordered keys alphabetically. It looks like an older PR accidentally committed an old schema.

Rails 8 changed the schema and ordered keys alphabetically. It looks like an older PR accidentally committed an old schema.
@cla-bot cla-bot Bot added the cla-signed label May 22, 2026
@zetter-rpf zetter-rpf marked this pull request as ready for review May 22, 2026 15:56
Copilot AI review requested due to automatic review settings May 22, 2026 15:56
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

Updates the committed Rails schema snapshot to match the current Rails 8.1 schema dump format, correcting a previously committed older-format schema.rb.

Changes:

  • Update schema header to ActiveRecord::Schema[8.1] for Rails 8.1.
  • Re-dump schema so extension declarations and table column/key ordering match Rails 8.1’s canonical/alphabetized output.

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

@github-actions
Copy link
Copy Markdown

Test coverage

91.29% line coverage reported by SimpleCov.
Run: https://github.com/RaspberryPiFoundation/editor-api/actions/runs/26298082485

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants