Skip to content
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

FI-3642 Updates wiki workflow to allow manual synchronization for previewing. #607

Merged
merged 2 commits into from
Jan 10, 2025

Conversation

arscan
Copy link
Contributor

@arscan arscan commented Jan 9, 2025

Allows users to 'preview' docs changes to the wiki, as in practice it is hard to ensure that docs changes look and work properly within the wiki which ends up creating a lot of junk PRs. Enables the workflow to be run manually via the Actions against any branch. Also automatically resynchs wiki docs to main on closure for any PR that has docs changes, just in case it was manually synchronized but not undone. (removed that run-on-close feature because it is spammy on merges, which counts as a PR closure)

Originally I created a workflow that would synchronize to the wiki automatically if a title contained something like '(DOCS)' in it.

But this requires people to know that this functionality exists. And at that point, it just seems simpler to allow manual execution of this via the workflow tab. So, the user opens a PR, goes to the Actions area, runs this against the new branch, which synchronizes the wiki. Takes a look. And then can resynchronize it back to main if they want, or let the reviewer take a look.

Less code, less magic stuff happening automatically.

I confirmed it work over here so I think this is a low risk merge with limited review: arscan/docs-test-kit#7.

@arscan arscan requested a review from Jammjammjamm January 9, 2025 21:35
@arscan arscan merged commit ed36384 into main Jan 10, 2025
3 checks passed
@arscan arscan deleted the update-workflow branch January 10, 2025 15:43
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.

2 participants