Skip to content

update.py: only copy target wiki files#7552

Open
khancyr wants to merge 1 commit intoArduPilot:masterfrom
khancyr:copy_wiki
Open

update.py: only copy target wiki files#7552
khancyr wants to merge 1 commit intoArduPilot:masterfrom
khancyr:copy_wiki

Conversation

@khancyr
Copy link
Copy Markdown
Contributor

@khancyr khancyr commented Mar 19, 2026

only copy file for the wiki we are building. this prevent uncessary remove and copy.

tested using
docker run --rm -it -v "$(pwd):/ardupilot_wiki" -u "$(id -u):$(id -g)" ardupilot_wiki python3 update.py --site copter --verbose
Output is the same as master

only the wiki selected files are copied

using
docker run --rm -it -v "$(pwd):/ardupilot_wiki" -u "$(id -u):$(id -g)" ardupilot_wiki python3 update.py --verbose

Output is the same as master
files for all wiki are copied

@Hwurzburg Hwurzburg requested a review from peterbarker March 19, 2026 11:21
Copy link
Copy Markdown
Contributor

@hunt0r hunt0r left a comment

Choose a reason for hiding this comment

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

Looks good. I left some optional suggestions, but they are only "if you like them". Feel free to resolve/ignore them all.

Comment thread update.py
Comment thread update.py
Comment thread update.py
Copy link
Copy Markdown
Contributor

@peterbarker peterbarker left a comment

Choose a reason for hiding this comment

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

Please fix the conflicts

@cclauss
Copy link
Copy Markdown
Contributor

cclauss commented Mar 21, 2026

% ruff check --select=UP031 --fix update.py # Should placate the linter

Comment thread update.py
Comment thread update.py
Comment thread update.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants