search

peterjc / mediawiki_to_git_md

Convert a MediaWiki export XML file into MarkDown as a series of git commits
MIT License
55 stars 17 forks source link

WIP Usage #42

Closed stappersg closed 4 months ago

stappersg commented 4 months ago

When #41 is accepted, I'll update this merge request.

Yes, I think it, #41 & #42, are two separate merge requests.

If you as project lead would have thmm in one request, just say so. I'll prepare 1 merge request with matching commit message. (Please avoid that github.com mangles my commit messages by squashing commits.)

peterjc commented 4 months ago

Is GitHub rebase and merge OK with you?

(I’m aware GitHub merge doesn’t respect all the original metadata)

Peter

On Mon, 15 Jul 2024 at 22:11, Geert Stappers @.***> wrote:

When #41 https://github.com/peterjc/mediawiki_to_git_md/pull/41 is accepted, I'll update this merge request.

Yes, I think it, #41 https://github.com/peterjc/mediawiki_to_git_md/pull/41 & #42 https://github.com/peterjc/mediawiki_to_git_md/pull/42, are two separate merge requests.

If you as project lead would have thmm in one request, just say so. I'll prepare 1 merge request with matching commit message. (Please avoid that github.com http://github.com mangles my commit messages by squashing commits.)

You can view, comment on, or merge this pull request online at:

https://github.com/peterjc/mediawiki_to_git_md/pull/42 Commit Summary

File Changes

(1 file https://github.com/peterjc/mediawiki_to_git_md/pull/42/files)

Patch Links:

— Reply to this email directly, view it on GitHub https://github.com/peterjc/mediawiki_to_git_md/pull/42, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAPTV2XKGEPW3A4NPN3PCTZMQ3JVAVCNFSM6AAAAABK5JXH6CVHI2DSMVQWIX3LMV43ASLTON2WKOZSGQYDSNRUHA2TMNQ . You are receiving this because you are subscribed to this thread.Message ID: @.***>

stappersg commented 4 months ago

Is GitHub rebase and merge OK with you?

I don't know. Increase the chance of avoiding mangled commits, please do the "Github rebase and merge" after merging of #41.