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

docs: rename Microsoft to microsoft #859

Merged
merged 1 commit into from
May 8, 2019

Conversation

markwolff
Copy link
Contributor

@markwolff markwolff commented May 8, 2019

rename travis and github links to solve issues from org rename to microsoft.

@@ -160,7 +160,7 @@ Parameter | Description
`handledAt` | Deprecated. This argument is ignored. Please pass `null`.
`properties` | Map of string to string: Additional data used to [filter exceptions](https://azure.microsoft.com/documentation/articles/app-insights-api-custom-events-metrics/#properties) in the portal. Defaults to empty.
`measurements` | Map of string to number: Metrics associated with this page, displayed in Metrics Explorer on the portal. Defaults to empty.
`severityLevel` | Supported values: [SeverityLevel.ts](https://github.com/Microsoft/ApplicationInsights-JS/blob/master/JavaScript/JavaScriptSDK.Interfaces/Contracts/Generated/SeverityLevel.ts)
`severityLevel` | Supported values: [SeverityLevel.ts](https://github.com/microsoft/ApplicationInsights-JS/blob/master/JavaScript/JavaScriptSDK.Interfaces/Contracts/Generated/SeverityLevel.ts)
Copy link
Member

Choose a reason for hiding this comment

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

Can you include more information about this change?

Copy link
Contributor Author

@markwolff markwolff May 8, 2019

Choose a reason for hiding this comment

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

The github organization was renamed from "Microsoft" to "microsoft", so this change is just a find&replace on this. Some things broke, like travis build badges

Copy link
Member

Choose a reason for hiding this comment

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

I got the actual change, my question is more related to why changing the org name if is going to break tons of people, how did you figure out the name was updated?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Other repos had similar PRs (e.g. microsoft/ApplicationInsights-Go#30) that showed up in notifcations. Unsure why the org change happened though.

@markwolff markwolff merged commit 329d77b into master May 8, 2019
@markwolff markwolff deleted the markwolff/lowercase_microsoft branch May 8, 2019 21:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants