-
Notifications
You must be signed in to change notification settings - Fork 1.5k
[Breadth Coverage] logic apps #1423
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.
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
Conversation
|
logic apps |
Copied from logic apps homepage. I think workflow.json is also a form of code. |
|
In The help messages provide nothing. They are same as parameter name. I don't know how to use it. What's the format of value? I also can't find usage in examples. |
|
Note sure how to fix the codeowner issue, I added myself to the .git/b
I could certainly add some more description here, but these parameters really do not lend themselves to a terse descriptions. For example, and integration service environment and account are connected to on premise subnets or Azure virtual networks that logic app can connect to to call either custom on premise services or custom services running on Azure. Logic apps are not something, I think, we should expect someone to try to use the CLI for without understanding these concepts before hand. This is part of the reason the logic app team has invested so much effort into their design tools. |
|
Not sure why a unrelated test is failing. |
There should be a button to require you to merge master if your branch do not have the latest master branch code. It's somehow gone. You can manually merge master and the issue of firewall should be fixed. |
Speclet: https://aka.ms/AA7txib
Files to focus on:
Since there are many generated files, the following are a list files that have been manually changed and hence should be the primary focus for the review:
This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines
azdev style <YOUR_EXT>locally? (pip install azdevrequired)python scripts/ci/test_index.py -qlocally?For new extensions:
About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update
src/index.jsonautomatically.The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify
src/index.json.