Skip to content

use esrp publishing #9189

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.

Sign up for GitHub

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

Jump to bottom
Draft
timotheeguerin wants to merge 48 commits into
base: main
Choose a base branch
from
Draft

use esrp publishing #9189

timotheeguerin wants to merge 48 commits into from
+415 −113

Conversation

@timotheeguerin
Copy link
Member

@timotheeguerin timotheeguerin commented Dec 11, 2025

No description provided.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 11, 2025 •
edited
Loading

❌ There is undocummented changes. Run chronus add to add a changeset or click here.

The following packages have changes but are not documented.

  • ❌@typespec/bundler
Show changes

@azure-sdk
Copy link
Collaborator

azure-sdk commented Dec 11, 2025 •
edited
Loading

You can try these changes here

🛝 Playground 🌐 Website 🛝 VSCode Extension

@pkg-pr-new
Copy link

pkg-pr-new bot commented Dec 12, 2025 •
edited
Loading

Open in StackBlitz

npm i https://pkg.pr.new/microsoft/typespec/@typespec/bundler@9189

commit: 2a06f18

timotheeguerin
timotheeguerin commented Dec 12, 2025
View reviewed changes
eng/tsp-core/pipelines/jobs/build-for-publish.yml
- script: pnpm run test:ci
displayName: Test
# - script: pnpm run test:ci
# displayName: Test
Copy link
Member Author

@timotheeguerin timotheeguerin Dec 12, 2025

Choose a reason for hiding this comment

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

revert when done

timotheeguerin
timotheeguerin commented Dec 12, 2025
View reviewed changes
eng/tsp-core/pipelines/publish.yml
# - template: /eng/tsp-core/pipelines/jobs/cli/build-tsp-cli-all.yml@self
# - template: /eng/tsp-core/pipelines/jobs/e2e.yml@self
# parameters:
# azLogin: true
Copy link
Member Author

@timotheeguerin timotheeguerin Dec 12, 2025

Choose a reason for hiding this comment

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

revert when done

timotheeguerin
timotheeguerin commented Dec 12, 2025
View reviewed changes
eng/tsp-core/pipelines/publish.yml
displayName: "Push"
condition: eq(variables['HAS_NEXT_PACKAGES_TO_PUBLISH'], 'true')

# - stage: publish_vscode
Copy link
Member Author

@timotheeguerin timotheeguerin Dec 12, 2025

Choose a reason for hiding this comment

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

revert when done

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

Reviewers

@bterlson bterlson Awaiting requested review from bterlson bterlson will be requested when the pull request is marked ready for review bterlson is a code owner

@markcowl markcowl Awaiting requested review from markcowl markcowl will be requested when the pull request is marked ready for review markcowl is a code owner

@witemple-msft witemple-msft Awaiting requested review from witemple-msft witemple-msft will be requested when the pull request is marked ready for review witemple-msft is a code owner

@chrisradek chrisradek Awaiting requested review from chrisradek chrisradek will be requested when the pull request is marked ready for review chrisradek is a code owner

@AlitzelMendez AlitzelMendez Awaiting requested review from AlitzelMendez AlitzelMendez will be requested when the pull request is marked ready for review AlitzelMendez is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

eng

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@timotheeguerin @azure-sdk