Skip to content

detect_tags_changes.py update doesn't work as I'd expect #199

@Timmmm

Description

@Timmmm

I renamed a tag in this PR: riscv/riscv-isa-manual#2646

If I run make build-tags then it does something very strange - it seems to merge the new name of the tag into ref/riscv-privileged-norm-tags.json, but it still leaves the old one there, and it's still listed in the section tree.

It seems like the code to do this is in this function:

    def update_tags_file(self, file_path: str, changes: TagChanges):
        """Update a tags file by adding new tags from additions.

        Args:
            file_path: Path to the file to update
            changes: Changes detected
        """

I don't understand why we'd ever want to merge stuff like this. All that should happen is it copies the new tags file over the old one. If you want to know what some old tag content was you can just check out an old version of the repo.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions