Skip to content

Move extensions projects to sdk/extensions directory#10053

Merged
pakrym merged 3 commits intoAzure:masterfrom
pakrym:pakrym/move-extensions
Feb 20, 2020
Merged

Move extensions projects to sdk/extensions directory#10053
pakrym merged 3 commits intoAzure:masterfrom
pakrym:pakrym/move-extensions

Conversation

@pakrym
Copy link
Copy Markdown
Contributor

@pakrym pakrym commented Feb 20, 2020

As discussed offline moving all extensions to the same location.

@pakrym pakrym requested a review from tg-msft February 20, 2020 01:03
@pakrym pakrym marked this pull request as ready for review February 20, 2020 01:03
@pakrym pakrym changed the title Move extensions projects to extensions directory Move extensions projects to sdk/extensions directory Feb 20, 2020
@pakrym pakrym merged commit ee8faec into Azure:master Feb 20, 2020
Comment thread sdk/extensions/ci.yml
- sdk/extensions/

stages:
- template: ../../eng/pipelines/templates/stages/archetype-sdk-client.yml
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What about other extensions like Microsoft.Extensions.Azure? Should they also be moved into here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, I'm planning to do that too. Maybe even rename it.

Comment thread sdk/extensions/tests.yml
@@ -0,0 +1,13 @@
trigger: none
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We have pipeline generation tooling that should automatically pick-up and setup a pipeline for the ci.yml file but we currently still have to manually setup the tests pipeline. Work with @danieljurek if you need help setting that up.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done.

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