Skip to content

Updated pattern maturity of patterns in markdown format #131

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

Merged
merged 4 commits into from
Mar 18, 2020
Merged

Updated pattern maturity of patterns in markdown format #131

merged 4 commits into from
Mar 18, 2020

Conversation

spier
Copy link
Member

@spier spier commented Mar 18, 2020

Fixes #128.

  • added/updated the "## Status" section for the patterns listed in README that are available as .md files
  • also ordered the sections according to the order in pattern-template.md.
  • fixed some minor typos too

Warning

Looks like my editor removed a bunch of spaces at the end of lines automatically?
I don't really know how that happened.

Unfortunately that makes the diff below a bit harder to read.

Other Observations

  • Prerequisites
    • found in: gig-marketplace.md, innersource-portal.md
    • not part of the Pattern template
  • "Solution"/"Solutions" / is used in singular and plural form in the patterns
  • "Author","Authors","Author(s)"
  • do we want newline after headlines in the .md files? Tiny difference admittedly :)
  • "Patlet" section was missing from some patterns. I added a placeholder for it. e.g. review-committee.md
  • sections that appear in some patterns but are not part of the Pattern template
    • See also
    • Related Patterns
    • Variants
    • Also Known As
  • having only one pattern in the folder "project-roles", while all other .md files are in the root folder is confusing

- Mostly added or update the "## Status" section for all patterns.
- also ordered the sections according to the order in `pattern-template.md`.
- fixed some minor typos too
@spier
Copy link
Member Author

spier commented Mar 18, 2020

@rrrutledge I am not done yet but I submitted this as a draft PR, so that you can see in which direction I am going with this.

Would you mind taking a look, so that I am not overdoing it?

Copy link
Contributor

@rrrutledge rrrutledge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great!

@rrrutledge
Copy link
Contributor

Everything that is there looks good.

spier added 3 commits March 18, 2020 19:15
…ewed)".

=> Maturity: Proven

Was actually just pattern `start-as-experiment.md`, as all other patterns in the block are still kept in gDocs, wiki, or open Pull Requests.
@spier spier marked this pull request as ready for review March 18, 2020 18:44
@spier
Copy link
Member Author

spier commented Mar 18, 2020

@rrrutledge I think that this is a good enough iteration i.e. I am happy with this to be merged, if you think the changes make sense.

@spier spier changed the title Updated all patterns in block "Reviewed Patterns (proven and reviewed)". Updated pattern maturity of patterns in markdown format Mar 18, 2020
This was referenced Mar 18, 2020
@rrrutledge rrrutledge merged commit a8ad8cd into InnerSourceCommons:master Mar 18, 2020
@spier spier deleted the issues/128-review-pattern-maturity-declaration branch March 19, 2020 06:05
@lenucksi lenucksi added 📖 Type - Content Work Working on contents is the main focus of this issue / PR Type - Maintenance / Cleanup Maintaining / cleaning the repo is the main focus of this issue / PR labels Sep 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📖 Type - Content Work Working on contents is the main focus of this issue / PR Type - Maintenance / Cleanup Maintaining / cleaning the repo is the main focus of this issue / PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Pattern Maturity in patterns does not match defined pattern-states nomenclature
3 participants