Skip to content

Update spring cloud again#1048

Merged
zikalino merged 7 commits intomasterfrom
update-spring-cloud-again
Oct 31, 2019
Merged

Update spring cloud again#1048
zikalino merged 7 commits intomasterfrom
update-spring-cloud-again

Conversation

@zikalino
Copy link
Copy Markdown


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

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:

@azuresdkci
Copy link
Copy Markdown

If this PR is for a new extension or change to an existing extension, use the following to try out the changes in this PR:

docker run -it microsoft/azure-cli:latest
export EXT=<NAME>
pip install --upgrade --target ~/.azure/cliextensions/$EXT "git+https://github.com/Azure/azure-cli-extensions.git@update-spring-cloud-again#subdirectory=src/$EXT&egg=$EXT"

@zikalino zikalino force-pushed the update-spring-cloud-again branch from b708213 to 5eeeb00 Compare October 31, 2019 11:22
"downloadUrl": "https://azurecliprod.blob.core.windows.net/cli-extensions/spring_cloud-0.1.0-py2.py3-none-any.whl",
"filename": "spring_cloud-0.1.0-py2.py3-none-any.whl",
"metadata": {
"classifiers": [
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.

No "azext.minCliCoreVersion": "2.0.46" needed here?

packages=find_packages(),
install_requires=DEPENDENCIES,
package_data={'azext_spring-cloud': ['azext_metadata.json']},
package_data={'azext_spring_cloud': ['azext_metadata.json']},
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.

package_data={'azext_ip-group': ['azext_metadata.json']},

Looks like this ext is also using -. Which one is correct?

@jiasli
Copy link
Copy Markdown
Member

jiasli commented Oct 31, 2019

We can merge for now. But the - and _ thing needs to be addressed asap.

@zikalino zikalino merged commit 811c0f4 into master Oct 31, 2019
@zikalino zikalino deleted the update-spring-cloud-again branch October 31, 2019 12:24
ManuInNZ pushed a commit to ManuInNZ/azure-cli-extensions that referenced this pull request Apr 11, 2020
* new index

* new spring-cloud index

* removed index

* another index

* regenerated index again

* fixed metadata

* fixed metadata?
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