Skip to content

Conversation

@Tom-van-Woudenberg
Copy link
Member

See https://github.com/TUDelft-MUDE/teacher/actions/runs/13992781818 for example

This:

  • allow you to move (next to copy and redirect) a branch built to the root, which prevents copying it. This shouldn't be standard behaviour but is in line with the desired behaviour for the MUDE-book
  • adapt the summary so that the table with branches shows the root url for the moved branch and the message at primary book at root is adapted

@Tom-van-Woudenberg Tom-van-Woudenberg self-assigned this Mar 21, 2025
@Tom-van-Woudenberg Tom-van-Woudenberg linked an issue Mar 21, 2025 that may be closed by this pull request
@Tom-van-Woudenberg Tom-van-Woudenberg added the enhancement New feature or request label Mar 21, 2025
@Tom-van-Woudenberg
Copy link
Member Author

Tom-van-Woudenberg commented Mar 21, 2025

@rlanzafame
Copy link
Member

To be honest I don't understand the reason why one would choose move over copy. Is it only to keep artifact size down and avoid duplication?

@Tom-van-Woudenberg
Copy link
Member Author

To be honest I don't understand the reason why one would choose move over copy. Is it only to keep artifact size down and avoid duplication?

Yes

Did I capture the move part OK?
I found a few other things to clean up/improve also, hence the PR.
@rlanzafame
Copy link
Member

See this PR #82
I did not set this up yet for PR into manual_docs

@Tom-van-Woudenberg Tom-van-Woudenberg merged commit 32e4f95 into main Mar 23, 2025
@Tom-van-Woudenberg Tom-van-Woudenberg deleted the move branch March 23, 2025 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow for move option

3 participants