Skip to content

Removing a branch from a book doesn't clear its metadata #973

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
nik9000 opened this issue Jul 9, 2019 · 0 comments · Fixed by #976
Closed

Removing a branch from a book doesn't clear its metadata #973

nik9000 opened this issue Jul 9, 2019 · 0 comments · Fixed by #976

Comments

@nik9000
Copy link
Member

nik9000 commented Jul 9, 2019

We use branch metadata to figure out if a book needs to be rebuilt. When you remove a branch from a book we don't clear the metadata. This leads to weird things like not automatically rebuilding books when the branch is re-added.

nik9000 added a commit that referenced this issue Jul 10, 2019
We keep information about which branches contribute to which books but
before this change we didn't automatically remove the metadata when
a branch is no longer involved in building a book we never cleared that
metadata. For the most part this just left lines in a YAML file that
aren't used except when we remove a branch from a book and then re-add
it. In that case since we only use the branch data to figure out if we
should rebuild a book we *wouldn't* start to build the book when it is
added back unless the branch that it is built from changes. This happened
to us recently so it makes sense to clean up these files automatically
so it never happens again.

Closes #973
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 a pull request may close this issue.

1 participant