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

Standardize capitalization of InnerSource#16

Merged
a-a-ron merged 7 commits intomasterfrom
cap-innersource
Apr 18, 2019
Merged

Standardize capitalization of InnerSource#16
a-a-ron merged 7 commits intomasterfrom
cap-innersource

Conversation

@a-a-ron
Copy link
Copy Markdown
Contributor

@a-a-ron a-a-ron commented Apr 17, 2019

In response to #15, this PR standardizes the capitalization of InnerSource across the course. The following were updated:

  • responses (also added updated final product image to welcome)
  • readme
  • config.yml (step descriptions)
  • template repo updated

closes #15

@a-a-ron a-a-ron requested a review from hectorsector April 17, 2019 16:04
Copy link
Copy Markdown
Contributor

@hectorsector hectorsector left a comment

Choose a reason for hiding this comment

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

Thanks @a-a-ron! I don’t know if changing the casing on the title will generate a new url for the course or not. When you merge this, could you see if the link to the course changes and, if so, update the tracking and feedback issues? https://github.com/github/services/issues/5527 and https://github.com/github/services/issues/5632

@a-a-ron a-a-ron merged commit d99ce7d into master Apr 18, 2019
@a-a-ron a-a-ron deleted the cap-innersource branch April 18, 2019 15:37
@a-a-ron
Copy link
Copy Markdown
Contributor Author

a-a-ron commented Apr 18, 2019

I don’t know if changing the casing on the title will generate a new url for the course or not

Links are good

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Standardize capitalization of InnerSource

2 participants