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

repo (mostly docs) cleanup #1667

Closed
bartekleon opened this issue Aug 8, 2019 · 5 comments
Closed

repo (mostly docs) cleanup #1667

bartekleon opened this issue Aug 8, 2019 · 5 comments
Labels
⁉ Question Further information is requested

Comments

@bartekleon
Copy link
Member

We should make some clean up repository, and possibly update documentation a bit.

Cleanup:

  • docs folder should be removed if it is not needed.

Documentation:
there is an issue in https://github.com/stryker-mutator/stryker/tree/master/packages/util
since download and node badges are not displayed as intended

Updating main README / somehow merging with core README. Since they are +- 1 to 1.
For example:

Getting started
For the quickstart, see quickstart.

Usage
For details on the stryker run command and others, see stryker's readme

Configuration
See the stryker's package readme for all config options.

Supported mutators
See our website for the list of currently supported mutators.

This part can be easy replaced by core README. Some tweaks might be needed tho

We should also consider making 1 unified README for stryker project instead of making tens of them (or at least reduce the amount).

Alternative option to the previous one is to make main README a bunch of links (like it is now) to our website, and making better docs on website (including all READMEs [or at least the most important parts]). This unfortunately will again increase number of unnesessary files.

@bartekleon bartekleon added the ⁉ Question Further information is requested label Aug 8, 2019
@gugacavalieri
Copy link
Contributor

Hey Guys!

I just took a quick look at the repo and I think all the docs moved to the handbook repo, am I right?
(https://github.com/stryker-mutator/stryker-handbook#readme)

And the docs folder in the main repo hasn't been updated in two years.

How are you handling documentation? Maybe I could give a hand with that 😄

@bartekleon
Copy link
Member Author

bartekleon commented Sep 26, 2019

thanks, @gugacavalieri for initiative! I would like @nicojs or @simondel to opinion that too, but for now, I think adding documentation for #1731 and #1623 would be nice :3. Also, it's a bit hard for me to tell where exactly docs are. Since half of them are in a handbook like you said, but also a big part of them is on the website https://github.com/stryker-mutator/stryker-mutator.github.io. So it's a bit messy in my opinion, but I can't do anything without their permission :/ (any idea, of course, is good!)

If you want to do something like right now, I guess you can always look for grammar / logic issues in docs. Since we are not native speakers (at least I am not), there can be plenty of mistakes

To sum up. Documentation is all over the place. You have a website, handbook, and readmes. I dunno where to find anything nor where to start :/

@bartekleon bartekleon changed the title repo cleanup repo (mostly docs) cleanup Sep 26, 2019
@gugacavalieri
Copy link
Contributor

gugacavalieri commented Oct 3, 2019

Cool ! @kmdrGroch

I will take a look at the docs then and check where the main part is also looking for some typos or grammar mistakes :)

It's feels kind of messy right now but I think we might be able to organize it!

Let's see what @nicojs and @simondel think as well

@bartekleon
Copy link
Member Author

bartekleon commented Oct 4, 2019

@gugacavalieri My main point is to change the amount of docs. We have docs in 3-4 different places and so it's messy and hard to maintain (just look on main repo doc)

Introduction
For an introduction to mutation testing and Stryker's features, see stryker-mutator.io.

Getting started
For the quickstart, see quickstart.

Usage
For details on the stryker run command and others, see stryker's readme

Configuration
See the stryker's package readme for all config options.

There are links to at least 2 places with +- the same docs

again core/readme has a line:
Please follow the quickstart on the website.

too much information in too many places. It can be messy sometimes ._.

In my opinion, we should have 1 overall doc file (1 doc place, it can be on website, so we can make some nice links, etc)

(it's also related to #1755 )

@gugacavalieri
Copy link
Contributor

@kmdrGroch I just took a quick look and the docs on the website are just a quick start.

Most of the technical details are in the core/readme file. So I think that is the one we should improve then.

Although sometimes the readme redirects the user to the website or to the handbook, maybe we could include all the info that is related to the core in this readme file and maybe start improving it with a Table of Contents section should be a good idea.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⁉ Question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants