Merge ~andrey-fedoseev/launchpad:db-remove-cve-discoverer into launchpad:db-devel

Proposed by Andrey Fedoseev
Status: Merged
Approved by: Andrey Fedoseev
Approved revision: a10d3fc608705f8d52ecabdf9a1bb549ff9e6f3d
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: ~andrey-fedoseev/launchpad:db-remove-cve-discoverer
Merge into: launchpad:db-devel
Diff against target: 15 lines (+9/-0)
1 file modified
database/schema/patch-2211-09-0.sql (+9/-0)
Reviewer Review Type Date Requested Status
William Grant (community) db Approve
Colin Watson (community) db Approve
Review via email: mp+430631@code.staging.launchpad.net

Commit message

Drop `Cve.discoverer` column

Description of the change

It is superseded by `Cve.discovered_by`

To post a comment you must log in.
Revision history for this message
Jürgen Gmach (jugmac00) wrote :

On its own I cannot approve this MP. Could you please link to the related MPs?

Revision history for this message
Andrey Fedoseev (andrey-fedoseev) wrote :

> On its own I cannot approve this MP. Could you please link to the related MPs?

Here you go https://code.launchpad.net/~andrey-fedoseev/launchpad/+git/launchpad/+merge/430020

Revision history for this message
Colin Watson (cjwatson) wrote :

This looks OK, but you can't land it on db-devel until your previous patch to stop using this column has been deployed to production.

Adding William for a second DB review.

review: Approve (db)
Revision history for this message
William Grant (wgrant) :
review: Approve (db)

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to status/vote changes: