Conversation
Co-authored-by: burisu <240595+burisu@users.noreply.github.com>
Co-authored-by: burisu <240595+burisu@users.noreply.github.com>
|
@copilot Things to change:
|
…orkflows Co-authored-by: burisu <240595+burisu@users.noreply.github.com>
All requested changes have been implemented in commit cdf1573:
Tests updated to verify both CSP formats and new routing structure. |
.github/workflows/test.yml
Outdated
| ruby-version: ['3.0', '3.1', '3.2'] | ||
| rails-version: ['6.1', '7.0', '7.1'] |
There was a problem hiding this comment.
Adjust versions to support Ruby >= 3.2 and Rails >= 7.1
There was a problem hiding this comment.
Updated in commit d82ec31. GitHub workflow now tests Ruby 3.2-3.3 and Rails 7.1-7.2. Also updated gemspec to require Ruby >= 3.2.0 and Rails >= 7.1.
| t.string :violated_directive | ||
| t.string :blocked_uri | ||
| t.text :original_policy | ||
| t.text :raw_report, null: false |
There was a problem hiding this comment.
Use JSONB instead of TEXT type for raw_report.
There was a problem hiding this comment.
Changed in commit d82ec31. Migration now uses t.jsonb :raw_report, null: false, default: {}. Controller updated to store hash directly instead of JSON string. All tests updated accordingly.
Co-authored-by: burisu <240595+burisu@users.noreply.github.com>
.github/workflows/test.yml
Outdated
|
@copilot Can you fix the tests too? |
Co-authored-by: burisu <240595+burisu@users.noreply.github.com>
Done in commit 4e09c83. Added rubocop-codeur as development dependency and normalized all Ruby code with:
|
Co-authored-by: burisu <240595+burisu@users.noreply.github.com>
Plan for implementing CSP reports collection Rails gem:
mountChanges made in this commit:
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.