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

Update documentation for module list #312

Merged
merged 2 commits into from
Jul 13, 2021
Merged

Conversation

VietThan
Copy link
Contributor

Also delete erroneous comment in KernelHelp module

VietThan added 2 commits July 11, 2021 21:42
KernelHelp comment mentioned POD format, probably erroneous
copy/paste
Following documentation style, added sections for
modules/format not yet written in README and in po4a.7.pod

Issue: mquinson#136
@mquinson
Copy link
Owner

Hello, many thanks for this contribution! That's a long awaited one, related to #136.

If I'm correct, you did not change all the lists that were pointed in #136. Is it that they were already uptodate, or are they still to be considered?

@mquinson mquinson merged commit 6fb1cda into mquinson:master Jul 13, 2021
@VietThan
Copy link
Contributor Author

Thanks @mquinson , I've changed all the ones in this repo. And yeah, I referenced the issue number with the pull request commit so GitHub will make that link automatically :D.
I'll make a separate pull for the one in the po4-website repo soon. I don't know but I don't think there is a way to do one commit, one pull request for two repos

@mquinson
Copy link
Owner

Indeed, it's impossible to have one PR applied to both repository, but for sake of simplicity, all bugs are counted on the po4a repo, even the ones applying to po4a-website. So I'm leaving #136 open until it gets fixed in po4a-website too.

Thanks for your time,

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