Skip to content

Commit 04549b8

Browse files
committed
Merge branch 'master' into rrrutledge-patch-3
2 parents 86f1468 + e470944 commit 04549b8

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

53 files changed

+1533
-419
lines changed

.github/CODEOWNERS

Lines changed: 40 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,40 @@
1+
# See GitHub documentation:
2+
# https://help.github.com/en/github/creating-cloning-and-archiving-repositories/about-code-owners
3+
4+
# This is a comment.
5+
# Each line is a file pattern followed by one or more owners.
6+
7+
# These owners will be the default owners for everything in
8+
# the repo. Unless a later match takes precedence,
9+
# @global-owner1 and @global-owner2 will be requested for
10+
# review when someone opens a pull request.
11+
* @lenucksi @nyeates @gruetter @NewMexicoKid @cewilliams
12+
13+
# Order is important; the last matching pattern takes the most
14+
# precedence. When someone opens a pull request that only
15+
# modifies JS files, only @js-owner and not the global
16+
# owner(s) will be requested for a review.
17+
# *.js @js-owner
18+
19+
# You can also use email addresses if you prefer. They'll be
20+
# used to look up users just like we do for commit author
21+
# emails.
22+
23+
24+
# In this example, @doctocat owns any files in the build/logs
25+
# directory at the root of the repository and any of its
26+
# subdirectories.
27+
# /build/logs/ @doctocat
28+
29+
# The `docs/*` pattern will match files like
30+
# `docs/getting-started.md` but not further nested files like
31+
# `docs/build-app/troubleshooting.md`.
32+
33+
34+
# In this example, @octocat owns any file in an apps directory
35+
# anywhere in your repository.
36+
# apps/ @octocat
37+
38+
# In this example, @doctocat owns any file in the `/docs`
39+
# directory in the root of your repository.
40+
# /docs/ @doctocat

.github/workflows/main.yml

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
# from: https://github.com/marketplace/actions/link-checker
2+
3+
name: Link Check on README.md
4+
5+
on: [push, pull_request]
6+
jobs:
7+
linkChecker:
8+
runs-on: ubuntu-latest
9+
steps:
10+
- uses: actions/checkout@v2
11+
- name: Link Checker
12+
id: lc
13+
uses: peter-evans/link-checker@v1
14+
with:
15+
args: -v -d . -x "http://creativecommons.org/licenses|https://isc-inviter.herokuapp.com" README.md
16+
- name: Fail if there were link errors
17+
run: exit ${{ steps.lc.outputs.exit_code }}

CONTRIBUTING.md

Lines changed: 18 additions & 74 deletions
Original file line numberDiff line numberDiff line change
@@ -1,95 +1,39 @@
1-
# How to Contribute
1+
# Contributing to the InnerSource Patterns
22

3-
First, checkout our complete list of patterns: [public GitHub](https://github.com/InnerSourceCommons/InnerSourcePatterns#reviewed-patterns-proven-and-reviewed) OR [Google Doc](https://docs.google.com/spreadsheets/d/17KPZdCoquTnYSj03pX4v2vn8lrSYO_6HK20u1cwaLPg/edit#gid=0)
3+
The goal of the InnerSource patterns working group is to collect, document, and publish InnerSource best practices. To make the best practices easy to comprehend, evaluate, and apply, we codify them in a specific structure - [the patterns format](meta/pattern-template.md). More infos on the working group can be found in our [README.md](README.md) file.
44

5-
We encourage beginners seeking answers to jump in by creating `donut` patterns (filling in the problem, context, forces and resulting context fields but leaving the solution blank) as a way of asking the InnerSource Commons community for help (to find a proven solution or to brainstorm things to try).
5+
For this working group to strive, we welcome your contribution - be it small or huge.
66

7-
Anyone can offer reviews and comments for [in-progress patterns](https://github.com/InnerSourceCommons/InnerSourcePatterns/pulls). We encourage experts to pad their experience - these are hoped to become part of an Inner Source handbook one day.
87

9-
We work together via GitHub, WebEx, Slack, etc. Do not hesitate to join the [#innersourcecommons](https://isc-inviter.herokuapp.com/) or #innersource-patterns Slack channels and ask to be included in the [patterns meetings](/meta/meetings.md) (there is an email list).
8+
## How to Contribute?
109

11-
Select one of the following ways to contribute, to learn our workflow:
10+
Please consider contributing in the following ways:
1211

13-
* A. [Write a new pattern](#a-writing-a-new-pattern)
14-
* B. [Discuss/Record early ideas](#b-discussing-early-ideas-in-issues)
15-
* C. [Review existing patterns](#c-interacting-with-patterns-reviews)
16-
* D. [Take part in our Meetings and Roles](/meta/meetings.md) <-- link to separate doc
12+
1. **Share your thoughts & spark discussion**. A fresh perspective, an interesting thought, or an early novel idea can spark discussions and yield unforeseeable results. We invite you to share your thoughts on InnerSource-related topics at any time by [creating an issue](https://github.com/InnerSourceCommons/InnerSourcePatterns/issues/new), [via Slack, or our patterns office hours](#how-to-get-in-touch).
1713

14+
2. **Read & improve existing patterns.** If you want to get started, we encourage you to read through the existing patterns and materials in this repository. Found a misleading phrase or phony grammar? A mistake in the content? Let us know by [creating an issue](https://github.com/InnerSourceCommons/InnerSourcePatterns/issues/new) or directly propose your fix as a pull request.
1815

19-
# A. Writing a New Pattern
16+
3. **Review open pull requests.** Even without knowing all the specifics of the patterns working group, your feedback on existing [pull requests](https://github.com/InnerSourceCommons/InnerSourcePatterns/labels/type%20-%20Content%20work) (especially those propising changes to pattern contents) will be valuable to its authors: As somebody who wants to learn more about InnerSource, you can give input from the perspective of a future reader. As somebody with own InnerSource experiences, you can share your experience with the authors.
2017

21-
The below steps can be used to create a new pattern. The use-case here is that you have an idea or problem in your head and can confidently fill out the barest of pattern fields (Solution doesn't need to be known). If you are unsure your idea is ready for this, [discuss it in an issue first](#b-discussing-early-ideas-in-issues).
18+
4. **Contribute a new pattern.** A key contribution you can make is to document an InnerSource best practice you experienced or know about. We defined different maturity levels for the contents in this repository: First time contributors should aim for maturity levels `1: initial` (for unstructured experience reports or what we call donuts - patterns with missing pieces) and `2: structured` (for best practices adhering to the patterns format and following basic writing style conventions). Our [Contributor Handbook](meta/contributor-handbook.md) discusses all this in more detail. You don't feel ready yet to contribute a new pattern? Please start a discussion by [creating an issue](https://github.com/InnerSourceCommons/InnerSourcePatterns/issues/new).
2219

23-
The simplest way to create a pattern is with your browser (see below).
24-
[Like the command-line better? See here for alternate instructions.](/meta/technical-git-howto.md)
20+
5. **Be a patterns advocate.** The patterns will become better and cover more topics the more people contribute their perspective and experience. A colleague told you about a novel InnerSource best practice? Please invite her/him to contribute to this repository!
2521

26-
<img alt="Creating a new pattern" src="/assets/img/write-new-pattern.png" width="70%">
2722

28-
1. Login to GitHub & inside the [patterns web repo](https://github.com/InnerSourceCommons/InnerSourcePatterns), click on the 'Create new file' button
29-
2. Name the file like this example: "project-management-time-pressures.md"
30-
3. Use the [pattern template](https://raw.githubusercontent.com/InnerSourceCommons/InnerSourcePatterns/master/meta/pattern-template.md) to create your new [markdown](/meta/markdown-info.md) file with the description of your fledgling pattern; it does not need to be complete, as you can add to it later
31-
4. Enter a commit message
32-
* If you are asked to 'Commit directly' vs 'Create a new branch', see [branching details](#use-branches-when-creating-new-patterns) below.
33-
5. Propose this new file and then also create a Pull Request (PR)
23+
## How to get in touch?
3424

35-
You're done! This creates a separate branch and creates a Pull Request (PR) all in one fell swoop! PR's are the mechanism we use for our Review process. See next steps in [Interacting with Pattern Reviews](#c-interacting-with-patterns-reviews).
25+
We are happy to support you in contributing to the InnerSource patterns or to just chat with you. You can reach out to us in the following ways:
3626

27+
1. **Via Slack.** Join the InnerSource Commons [Slack instance](https://isc-inviter.herokuapp.com/) and enter the `#innersource-patterns` channel there.
3728

38-
## Use Branches When Creating New Patterns
39-
We develop new patterns in branches with the naming convention:
40-
`pattern/<title-of-pattern-here>`.
29+
2. **During the patterns office hours.** We established a regular cadence of office hours for you to ask questions. You can find out about the next patterns office hours in the calendar at the bottom of [this page](https://innersourcecommons.org/resources/).
4130

42-
If you are asked to 'Commit directly...' vs 'Create a new branch...'
4331

44-
* Assure you select **'Create a new branch...'** and name the branch like this example: "pattern/project-management-time-pressures".
45-
* This occurs when writing a new pattern via the web interface (section A above).
46-
* Only [Trusted Committers](/meta/trusted-committers.md) (TC's) are asked this; we are adding most contributors as TC's.
32+
## License of Contributions
4733

34+
The contents of this repository are licensed under [CC-BY-SA-4.0](LICENSE.md). By contributing to this repository, you grant us (and everyone else for that matter) the right to use your contribution in accordance with that license.
4835

49-
## Other Tips For Submissions
5036

51-
* Place each sentence on a new line.
52-
_GitHub_ allows leaving comments on a line-by-line basis.
53-
Review and comment on the content of submitted text is much easier if there are multiple lines on-which to leave comments.
54-
Sentences on consecutive lines will be collapsed into a single paragraph (like this one) for the final reader of the content.
37+
## Code of Conduct
5538

56-
# B. Discussing Early Ideas in Issues
57-
58-
If you feel that you need extra advice when dealing with patterns, please [open an issue](https://github.com/InnerSourceCommons/InnerSourcePatterns/issues). This process is only needed when contributing early ideas that you are uncertain about.
59-
60-
Here are tips on starting this discussion:
61-
62-
* [Create a new ticket](https://github.com/InnerSourceCommons/InnerSourcePatterns/issues/new), add a concise title, and describe your problem. Think about the context of your problem and your expected output. Where do you see this problem most? What is the setup of your business and organization? Do you have opinions/ideas on what causes or leads-to the problem?
63-
* Ask any questions that you are unsure about. Are you unsure if this problem belongs here? Are you unsure on how to frame and explain the problem?
64-
* [Apply the label](https://help.github.com/articles/applying-labels-to-issues-and-pull-requests/) `Early Idea`. Labels can be found in the right column settings.
65-
66-
After this process, it is our turn to drive you through the pattern creation process. We will help to land your idea and check if there are existing similar patterns.
67-
68-
69-
70-
# C. Interacting with Patterns Reviews
71-
72-
A pattern is said to be "in-review" or being "Reviewed" when we have a Pull Request with some amount of Pattern detail filled out. We then communally review, and comment-on, and OK these "in-review" patterns. Usually, we first look for a pattern with all the fields filled out, and then go through TWO peer-reviews.
73-
74-
## Editing a pattern that is in-review
75-
Go ahead, edit away - we can always go back - and we encourage action over discussion.
76-
77-
<img alt="How to edit an existing pattern" src="/assets/img/edit-existing-pattern.png" width="70%">
78-
79-
## Reviewing a pattern
80-
FIXME Explain how to add review comments and accepting a review. Basically, this is all done through Githubs web GUI around Pull Requests.
81-
82-
FIXME Give tips for good reviews. We have done both interspersed comments, or pattern-wide advise. Be constructive. If you can fix the problem, [edit the PR](#editing-a-pattern-that-is-in-review) instead of leaving a comment.
83-
84-
## Our Review Process
85-
Below are the procedural steps in our Review process:
86-
87-
1. Decide which Maturity level your pattern is in: `Donut (Lacks solution)`, `Unproven`, or `Proven`; these all describe what state the *Solution* is in.
88-
2. Decide which Review Step you are in: Usually `Incomplete` or `Do 1st Review`
89-
3. Reviewers can now use the PR features to comment on the pattern.
90-
4. In most cases, we do two reviews, and the PR's labels should reflect `Do 2nd Review` etc.
91-
5. After reviews are complete, the reviewers or author should Revise and Finalize the pattern, eventually labeling it with `Accepted`.
92-
6. Once a pattern is `Accepted` by the reviewers, one of the [Trusted Committers](/meta/trusted-committers.md) (most authors are by this point) can Merge the PR on Github. This places the .md file into the master branch / root directory.
93-
94-
## Completed Patterns
95-
When completed patterns (reviewed and accepted) are ready to be published from this InnerSourcePatterns repo to a Gitbook (PDF), [see our separate Publishing instructions](/meta/publishing.md).
39+
We do not yet have a fully formalized code of conduct yet. We expect all contributors to treat each other respectfully and help us to keep this community free of harassment and discrimination.

0 commit comments

Comments
 (0)