Skip to content

[Azure Search] Branding changes to Azure Cognitive Search#8316

Merged
arv100kri merged 2 commits into
Azure:masterfrom
bleroy:beleroy/branding-azure-cognitive-search
Oct 22, 2019
Merged

[Azure Search] Branding changes to Azure Cognitive Search#8316
arv100kri merged 2 commits into
Azure:masterfrom
bleroy:beleroy/branding-azure-cognitive-search

Conversation

@bleroy
Copy link
Copy Markdown
Contributor

@bleroy bleroy commented Oct 22, 2019

This PR integrates changes already done to the specs under the generated changeset, as well as new changes to the non-generated classes. These changes are only in doc comments, never in code.

@isra-fel isra-fel added Mgmt This issue is related to a management package. needs-review labels Oct 22, 2019
@isra-fel isra-fel self-assigned this Oct 22, 2019
@isra-fel isra-fel removed the Mgmt This issue is related to a management package. label Oct 22, 2019
@isra-fel
Copy link
Copy Markdown
Member

The management plane changes look good to me. Just a reminder: you'll need another PR to bump up versions (if you'd like to publish a new version)

@arv100kri
Copy link
Copy Markdown
Member

@bleroy We shouldn’t make the management plane changes!! The responsibility there is with Mike’s team. we only own and ship the data plane

@bleroy bleroy force-pushed the beleroy/branding-azure-cognitive-search branch from e45a45f to e0a81ce Compare October 22, 2019 17:50
@arv100kri arv100kri changed the title Branding changes to Azure Cognitive Search [Azure Search] Branding changes to Azure Cognitive Search Oct 22, 2019
@arv100kri
Copy link
Copy Markdown
Member

The management plane changes look good to me. Just a reminder: you'll need another PR to bump up versions (if you'd like to publish a new version)

We have reverted the changes to the management plane side. And yes, we’ll have a separate PR to bump the version and changelog for a release

@arv100kri arv100kri merged commit f944d62 into Azure:master Oct 22, 2019
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.

3 participants