Skip to content

Conversation

@anjbur
Copy link
Contributor

@anjbur anjbur commented Sep 28, 2022

PR for az quantum extension, providing updates for resource estimation.


This checklist is used to make sure that common guidelines for a pull request are followed.

Related command

General Guidelines

  • Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
  • Have you run python scripts/ci/test_index.py -q locally?

For new extensions:

About Extension Publish

There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
The precondition is to put your code inside this repository and upgrade the version in the pull request but do not modify src/index.json.

@ghost ghost added the Auto-Assign Auto assign by bot label Sep 28, 2022
@ghost ghost requested review from kairu-ms and necusjz September 28, 2022 21:23
@ghost ghost assigned kairu-ms Sep 28, 2022
@ghost ghost added this to the Sep 2022 (2022-10-12) - For Ignite milestone Sep 28, 2022
@ghost ghost requested a review from yonzhan September 28, 2022 21:23
@ghost ghost added the Quantum az quantum label Sep 28, 2022
@ghost ghost requested a review from wangzelin007 September 28, 2022 21:23
@yonzhan
Copy link
Collaborator

yonzhan commented Sep 28, 2022

Quantum

@kairu-ms
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

return

if job.target.startswith("microsoft.simulator"):
if job.target.startswith("microsoft.simulator") and job.target != "microsoft.simulator.resources-estimator":
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this the correct name?

Copy link
Contributor

Choose a reason for hiding this comment

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

yes I think we changed the sku name, but not the target name

@shenyingjun
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 5399 in repo Azure/azure-cli-extensions

@anjbur
Copy link
Contributor Author

anjbur commented Oct 10, 2022

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 5399 in repo Azure/azure-cli-extensions

@kairu-ms kairu-ms merged commit 7df19a7 into Azure:main Oct 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Auto-Assign Auto assign by bot Quantum az quantum

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants