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

Review all README/documentation to remove outdated/duplicate content #15

Open
jmdh opened this issue Oct 13, 2017 · 9 comments
Open

Review all README/documentation to remove outdated/duplicate content #15

jmdh opened this issue Oct 13, 2017 · 9 comments

Comments

@jmdh
Copy link
Member

jmdh commented Oct 13, 2017

This file is severely out of date, but still probably contains useful snippets. It should probably be reviewed and merged with the top-level README.

@Tux
Copy link
Collaborator

Tux commented Oct 15, 2017

I updated most of the serious historic flaws. Karl is testing the current docs to the letter in doing exactly what it states and changing the docs along the way. Once done (I sit next to Karl to verify or comment on every change) I'll ask Jim to convert the README to MarkDown and improve on style and consistency

@jmdh
Copy link
Member Author

jmdh commented Oct 17, 2017

for the avoidance of doubt, U/README.dist hasn't been touched since 2011 (aside from the small change to fix a specific error from me earlier). There's also U/README which again needs an overhaul/removal. I'll retitle this ticket accordingly. (+ @jkeenan as you were mentioned above)

@jmdh jmdh changed the title Update U/README.dist Review all README/documentation to remove outdated/duplicate content Oct 17, 2017
@khwilliamson
Copy link
Contributor

I have issued PR13 for fixing this up.

One thing I mostly didn't change is the instructions for (k) Porting/mksample. I ran this and immediately regretted it because the changes were so vast from what came before. I think Tux should figure this out.

Also, in step (a). This is so confusing to me that i don't know how to clarifiy it.

@jkeenan
Copy link
Contributor

jkeenan commented Oct 18, 2017 via email

@khwilliamson
Copy link
Contributor

Also under (k), I don't know what it means to make sure all the developers are happy. Does that mean I'm supposed to buy them a round?

@khwilliamson
Copy link
Contributor

And I don't remember what Tux said about these lines

(i) Check if U/mkglossary (right near the top) points to where you keep
dist's standard metaconfig units as well as your perl-specific ones.

But I don't understand them; perhaps this is why the glossary isn't getting made.

@Tux
Copy link
Collaborator

Tux commented Oct 19, 2017

I will re-visit step (a)
@jkeenan I have no objections to you changing README to README.md now that @khwilliamson and I have changed the main flow to current status. Shall I create a new issue and assign that to you?

@jkeenan
Copy link
Contributor

jkeenan commented Oct 20, 2017 via email

@Tux
Copy link
Collaborator

Tux commented Nov 12, 2017

Does commit 0a071d9 address the confusin you have/had?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants