Skip to content

Comments

feat: rename labels#1858

Merged
Myriad-Dreamin merged 2 commits intomainfrom
rename-label
Aug 9, 2025
Merged

feat: rename labels#1858
Myriad-Dreamin merged 2 commits intomainfrom
rename-label

Conversation

@Myriad-Dreamin
Copy link
Owner

closes #549

@Myriad-Dreamin Myriad-Dreamin changed the title Rename label feat: rename labels Jun 30, 2025
@Myriad-Dreamin Myriad-Dreamin requested a review from Copilot June 30, 2025 03:02
Copy link
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

This PR refactors the prepare_renaming match to separate TODOs for bibkey and label renaming into distinct arms.

  • Extracted BibEntry(..) into its own arm with a bibkey TODO.
  • Grouped Label(..) and ContentRef(..) into a separate arm with a label TODO.
  • Preserved existing arms for other node types.

@Myriad-Dreamin Myriad-Dreamin marked this pull request as draft June 30, 2025 03:17
@Myriad-Dreamin Myriad-Dreamin marked this pull request as ready for review August 9, 2025 10:41
@Myriad-Dreamin Myriad-Dreamin merged commit 14c7a92 into main Aug 9, 2025
42 checks passed
@Myriad-Dreamin Myriad-Dreamin deleted the rename-label branch August 11, 2025 21:32
Myriad-Dreamin added a commit that referenced this pull request Aug 26, 2025
Myriad-Dreamin added a commit that referenced this pull request Aug 26, 2025
This reverts commit 14c7a92. Didn't
pass the test `label_indir.typ.snap`.
Myriad-Dreamin added a commit that referenced this pull request Aug 26, 2025
This reverts commit 856a1e4 (#2084). Didn't pass the test `label_indir.typ.snap`.
Myriad-Dreamin added a commit that referenced this pull request Dec 8, 2025
This reverts commit 856a1e4 (#2084). Didn't pass the test `label_indir.typ.snap`.
Myriad-Dreamin added a commit that referenced this pull request Jan 3, 2026
Rework on the reverted commit 856a1e4
(#2084). Didn't pass the test `label_indir.typ.snap`.
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.

Rename labels

1 participant