Skip to content
This repository was archived by the owner on Sep 1, 2022. It is now read-only.

Commit 75ce2b5

Browse files
authored
Merge pull request #11 from githubtraining/response-tidy
Standardize responses
2 parents 43519f9 + 18e7c93 commit 75ce2b5

15 files changed

Lines changed: 47 additions & 44 deletions

responses/00_welcome.md

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,15 @@
1-
This repository will eventually be a working web site that you'll be able to publish internally as your own Innersource commons.
1+
# Welcome!
2+
3+
This repository will be a working website that you can publish as your own Innersource commons.
24

35
![screenshot of final product](https://user-images.githubusercontent.com/16547949/56064057-f1f17700-5d3e-11e9-9100-9678dc91c544.png)
46

5-
As we work on this repo, we'll be creating a resource others can use within your organization. This repo can continue to be used after the course is finished.
7+
As we work, we'll create a resource that others can use within your organization. This repository will be useful even after you finish this course.
68

79
This is how this course will move along:
8-
- every new issue or pull request will start with a challenge question
9-
- you'll answer the challenge question; don't worry, there are no right or wrong answers
10-
- the question will highlight why each topic is an important part of applying Innersource practices right
10+
- Every new issue or pull request will start with a challenge question
11+
- You'll answer the challenge question _(don't worry, there are no right or wrong answers!)_
12+
- The question will highlight the importance of each topic in Innersource practices
1113

1214
Here's your first challenge question!
1315

responses/01_innersource-v-opensource.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
Innersource and open source are very similar concepts, but they're not the same.
44

5-
**Open source** software is shared publicly and typically accepts contributions from contributors outside of an organization.
5+
**Open source** software public and typically accepts contributions from outside of an organization.
66

77
**Innersource** is the practice of adopting open source practices internally within your organization. An organization that practices innersource, may or may not end up open sourced.
88

responses/01_repo-owner.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@ This pull request is all about repository ownership and naming conventions.
44

55
Does a repository exist at https://github.com/githubtraining/training-manual?
66

7-
- [ ] True, there is a repository at the given URL.
8-
- [ ] False, no repository exists at the given URL.
7+
- [ ] True, there is a repository at the given URL
8+
- [ ] False, no repository exists at the given URL
99

1010
<hr>
1111
<h3 align="center">I'll respond when you check one of the boxes above.</h3>

responses/02_repo-owner-answer.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,15 +7,15 @@ A repository _does_ exist at githubtraining/training-manual, but it is private.
77
<details>
88
<summary>Why can't you see it then? Review the <b>Files Changed</b> tab and see if you can figure it out. Collapse this text if you'd like an explanation.</summary>
99

10-
The repository `training-manual` in the @githubtraining organization is private, and only visible to memebers of that organization. If you're not a member of the organization, it'll appear like the repository doesn't exist to you. This is an example of why having the right ownership structure is important. Having too many organizations with restrictive permissions means that each organization's work will be siloed.
10+
The repository `training-manual` in the @githubtraining organization is private. It's only visible to members of that organization. If you're not a member of the organization, it'll appear like the repository doesn't exist to you. This is an example of why having the right ownership structure is important. Having too many organizations with restrictive permissions silos and isolates each organization's work.
1111
</summary>
1212

1313
Here are some recommendations based on some :sparkles: use of GitHub that we've seen:
14-
- aim for as few organizations as possible, remember: each organization is a black box to those outside of that organization
15-
- name your repositories in a meaningful manner, usually a simple project or application name will suffice
14+
- Aim for as few organizations as possible. Remember, each organization is a black box to those outside of that organization.
15+
- Name your repositories in a meaningful manner. Usually a simple project or application name will suffice.
1616
</summary>
1717

18-
Please merge this PR.
18+
Please merge this pull request.
1919

2020
<hr>
2121
<h3 align="center">I'll respond when you merge this pull request.</h3>

responses/03_discoverable.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
This PR is all about how to make your repositories more discoverable.
1+
This pull request is all about how to make your repositories more discoverable.
22

33
### Challenge question
44

responses/04_discoverability-answer.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,13 +5,13 @@ This repository is the source code for https://services.github.com.
55
Did you guess it? Probably not, because it doesn't contain many of the necessary elements to make a repository discoverable.
66

77
Some suggestions to ensure your teammates can find your repository are:
8-
- use a descriptive repository name
9-
- give your repository a description
10-
- provide a URL for the deployment of the app, or documentation about this project
8+
- Use a descriptive repository name
9+
- Give your repository a description
10+
- Provide a URL for the deployment of the app, or documentation about this project
1111

1212
Discover more about how to make a repository discoverable by visiting the **Files Changed** tab. It's OK if you want to do this later, any resources we cover throughout the course will be available at the end.
1313

14-
Please merge this PR.
14+
Please merge this pull request.
1515

1616
<hr>
1717
<h3 align="center">I'll respond when you merge this pull request.</h3>

responses/05_templates.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ Here are four different ways you could submit a bug report. Which is the _right_
77
- [ ] Open an issue on GitHub
88
- [ ] Create a Jira issue
99
- [ ] Open an internal support ticket
10-
- [ ] DM or email the maintainer
10+
- [ ] Direct message or email the maintainer
1111

1212
<hr>
1313
<h3 align="center">I'll respond when you check one of the boxes above.</h3>

responses/06_templates-answer.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,9 @@
22

33
Every project _can_ have a different way to submit bug reports, so all the answers are right.
44

5-
But, how could you know? With issue and pull request templates, you can take the mystery out of the equation and allow anyone across the organization to know the way each project prefers to receive issues and contributions.
5+
But, how could you know? With issue and pull request templates, you can take the mystery out of the equation. Templates show those across the organization the preferred way to contribute.
66

7-
Please merge this PR.
7+
Please merge this pull request.
88

99
<hr>
1010
<h3 align="center">I'll respond when you merge this pull request.</h3>

responses/07_workflows.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
1-
This PR covers a few common workflows that are likely in use within your organization.
1+
This pull request covers a few common workflows that are likely in use within your organization.
22

33
### Challenge question
44

5-
Suppose you'd like to contribute to the repository whose branches are pictured below. Which branch should I use as a base for my contributions?
5+
Suppose you'd like to contribute to the repository pictured below. Which branch should I use as a base for my contributions?
66

77
![a screenshot of a repository's branches, they are: a protected master branch, a protected release-v1.0 branch. The rest are unprotected: bug-fix, green-colors, config-release-drafter, add-template](https://user-images.githubusercontent.com/16547949/56039935-5d1d5800-5d03-11e9-8a26-29d36cfc7499.png)
88

responses/08_workflows-answer.md

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,13 +2,13 @@
22

33
The real answer is that, without documentation, we don't know!
44

5-
This PR covers the basics, but here are some recommendations to help any contributor know the right way to open pull requests:
6-
- use protected branches, doing so will allow you to accept PRs without fear of breaking production code
7-
- use Codeowners in combination with protected branches, to ensure that the right people are pinged for specific changes
8-
- if you haven't picked a workflow, use a common and well documented one that your contributors can easily search for
9-
- be explicit about your deployment and release strategy
5+
This pull request covers the basics, but here are some recommendations to help any contributor know the right way to open pull requests:
6+
- Use protected branches. Doing so will allow you to accept pull requests without fear of breaking production code.
7+
- Use `CODEOWNERS` in combination with protected branches to ensure the right people review changes.
8+
- If you haven't picked a workflow, use a common and well documented one.
9+
- Be explicit about your deployment and release strategy.
1010

11-
Please merge this PR.
11+
Please merge this pull request.
1212

1313
<hr>
1414
<h3 align="center">I'll respond when you merge this pull request.</h3>

0 commit comments

Comments
 (0)