Skip to content

Drop svn_root and replace todolists with pkgbases#663

Merged
jelly merged 1 commit into
archlinux:masterfrom
benmezger:refactor/rename-svn-to-git
May 14, 2026
Merged

Drop svn_root and replace todolists with pkgbases#663
jelly merged 1 commit into
archlinux:masterfrom
benmezger:refactor/rename-svn-to-git

Conversation

@benmezger
Copy link
Copy Markdown
Contributor

@benmezger benmezger commented Apr 11, 2026

PR drops svn_root column, as its now meaningless with our Git repository workflow.

This could potentially be a breaking change in case another application that reads the same database relies on the svn_root column or some external API depends on that name.

Comment thread main/models.py Outdated
@benmezger benmezger marked this pull request as ready for review April 11, 2026 11:40
Copy link
Copy Markdown
Member

@jelly jelly left a comment

Choose a reason for hiding this comment

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

I think its better to drop svn_root altogether and replace it in the todo's with a link to pkgbases. What might be better to split this into core/extra but honestly lets make that a follow up.

@benmezger benmezger force-pushed the refactor/rename-svn-to-git branch from 53d96d1 to 0b5def5 Compare April 11, 2026 15:31
@benmezger benmezger changed the title Rename svn_root -> git_root Drop svn_root and replace todolists with pkgbases Apr 11, 2026
@benmezger benmezger requested a review from jelly April 14, 2026 07:49
Copy link
Copy Markdown
Member

@jelly jelly left a comment

Choose a reason for hiding this comment

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

Just needs some small minor fixes and its good to merge.

Comment thread main/fixtures/repos.json
Comment thread templates/todolists/view.html
@jelly jelly force-pushed the refactor/rename-svn-to-git branch from 0b5def5 to 3c85de4 Compare May 14, 2026 13:40
@jelly jelly merged commit 6ec23f5 into archlinux:master May 14, 2026
1 check passed
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.

2 participants