Skip to content

Commit

Permalink
Merge pull request #47 from jddarby/jdd/multi-nf-support
Browse files Browse the repository at this point in the history
multi nf support
  • Loading branch information
jamiedparsons authored Jul 27, 2023
2 parents c087b45 + 687d506 commit b192027
Show file tree
Hide file tree
Showing 46 changed files with 7,216 additions and 557 deletions.
14 changes: 0 additions & 14 deletions src/aosm/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -198,17 +198,3 @@ az config set logging.enable_log_file=false

## Development
Information about setting up and maintaining a development environment for this extension can be found [here](./development.md).

## Linting
Please run mypy on your changes and fix up any issues before merging.
```bash
cd src/aosm
mypy . --ignore-missing-imports --no-namespace-packages --exclude "azext_aosm/vendored_sdks/*"
```

## Pipelines
The pipelines for the Azure CLI run in ADO, not in github.
To trigger a pipeline you need to create a PR against main.
Until we do the initial merge to main we don't want to have a PR to main for every code review.
Instead we have a single PR for the `add-aosm-extension` branch: https://github.com/Azure/azure-cli-extensions/pull/6426
Once you have merged your changes to `add-aosm-extension` then look at the Azure Pipelines under https://github.com/Azure/azure-cli-extensions/pull/6426/checks, click on the link that says `<X> errors / <Y> warnings`.
Loading

0 comments on commit b192027

Please sign in to comment.