-
Notifications
You must be signed in to change notification settings - Fork 622
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
'Updated AL-Go System Files' #20892
Merged
Merged
'Updated AL-Go System Files' #20892
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mazhelez
requested review from
JesperSchulz,
darjoo,
IhorHandziuk,
stkillen and
AndreasMoth
as code owners
November 7, 2022 21:54
JesperSchulz
approved these changes
Nov 8, 2022
AMCJN
pushed a commit
to AMCJN/ALAppExtensions
that referenced
this pull request
Nov 16, 2022
## Preview Note that when using the preview version of AL-Go for GitHub, you need to Update your AL-Go system files, as soon as possible when told to do so. ### Issues - Issue microsoft#233 AL-Go for GitHub causes GitHub to issue warnings - Issue microsoft#244 Give error if AZURE_CREDENTIALS contains line breaks ### Changes - New workflow: PullRequestHandler to handle all Pull Requests and pass control safely to CI/CD - Changes to yaml files, PowerShell scripts and codeowners files are not permitted from fork Pull Requests - Test Results summary (and failed tests) are now displayed directly in the CI/CD workflow and in the Pull Request Check ### Continuous Delivery - Proof Of Concept Delivery to GitHub Packages and Nuget ## v2.0 ### Issues - Issue microsoft#143 Commit Message for **Increment Version Number** workflow - Issue microsoft#160 Create local DevEnv aith appDependencyProbingPaths - Issue microsoft#156 Versioningstrategy 2 doesn't use 24h format - Issue microsoft#155 Initial Add existing app fails with "Cannot find path" - Issue microsoft#152 Error when loading dependencies from releases - Issue microsoft#168 Regression in preview fixed - Issue microsoft#189 Warnings: Resource not accessible by integration - Issue microsoft#190 PublishToEnvironment is not working with AL-Go-PTE@preview - Issue microsoft#186 AL-GO build fails for multi-project repository when there's nothing to build - When you have GitHub pages enabled, AL-Go for GitHub would try to publish to github_pages environment - Special characters wasn't supported in parameters to GitHub actions (Create New App etc.) ### Continuous Delivery - Added new GitHub Action "Deliver" to deliver build output to Storage or AppSource - Refactor CI/CD and Release workflows to use new deliver action - Custom delivery supported by creating scripts with the naming convention DeliverTo*.ps1 in the .github folder ### AppSource Apps - New workflow: Publish to AppSource - Continuous Delivery to AppSource validation supported ### Settings - New Repo setting: CICDPushBranches can be specified as an array of branches, which triggers a CI/CD workflow on commit. Default is main', release/\*, feature/\* - New Repo setting: CICDPullRequestBranches can be specified as an array of branches, which triggers a CI/CD workflow on pull request. Default is main - New Repo setting: CICDSchedule can specify a CRONTab on when you want to run CI/CD on a schedule. Note that this will disable Push and Pull Request triggers unless specified specifically using CICDPushBranches or CICDPullRequestBranches - New Repo setting: UpdateGitHubGoSystemFilesSchedule can specify a CRONTab on when you want to Update AL-Go System Files. Note that when running on a schedule, update AL-Go system files will perfom a direct commit and not create a pull request. - New project Setting: AppSourceContext should be a compressed json structure containing authContext for submitting to AppSource. The BcContainerHelperFunction New-ALGoAppSourceContext will help you create this structure. - New project Setting: AppSourceContinuousDelivery. Set this to true in enable continuous delivery for this project to AppSource. This requires AppSourceContext and AppSourceProductId to be set as well - New project Setting: AppSourceProductId should be set to the product Id of this project in AppSource - New project Setting: AppSourceMainAppFolder. If you have multiple appFolders, this is the folder name of the main app to submit to AppSource. ### All workflows - Support 2 folder levels projects (apps\w1, apps\dk etc.) - Better error messages for if an error occurs within an action - Special characters are now supported in secrets - Initial support for agents running inside containers on a host - Optimized workflows to have fewer jobs ### Update AL-Go System Files Workflow - workflow now displays the currently used template URL when selecting the Run Workflow action ### CI/CD workflow - Better detection of changed projects - appDependencyProbingPaths did not support multiple projects in the same repository for latestBuild dependencies - appDependencyProbingPaths with release=latestBuild only considered the last 30 artifacts - Use mutex around ReadSecrets to ensure that multiple agents on the same host doesn't clash - Add lfs when checking out files for CI/CD to support checking in dependencies - Continue on error with Deploy and Deliver ### CI/CD and Publish To New Environment - Base functionality for selecting a specific GitHub runner for an environment - Include dependencies artifacts when deploying (if generateDependencyArtifacts is true) ### localDevEnv.ps1 and cloudDevEnv.ps1 - Display clear error message if something goes wrong Co-authored-by: mazhelez <[email protected]>
aholstrup1
pushed a commit
to aholstrup1/ALAppExtensions
that referenced
this pull request
Sep 6, 2024
## Preview Note that when using the preview version of AL-Go for GitHub, you need to Update your AL-Go system files, as soon as possible when told to do so. ### Issues - Issue #233 AL-Go for GitHub causes GitHub to issue warnings - Issue #244 Give error if AZURE_CREDENTIALS contains line breaks ### Changes - New workflow: PullRequestHandler to handle all Pull Requests and pass control safely to CI/CD - Changes to yaml files, PowerShell scripts and codeowners files are not permitted from fork Pull Requests - Test Results summary (and failed tests) are now displayed directly in the CI/CD workflow and in the Pull Request Check ### Continuous Delivery - Proof Of Concept Delivery to GitHub Packages and Nuget ## v2.0 ### Issues - Issue #143 Commit Message for **Increment Version Number** workflow - Issue #160 Create local DevEnv aith appDependencyProbingPaths - Issue #156 Versioningstrategy 2 doesn't use 24h format - Issue #155 Initial Add existing app fails with "Cannot find path" - Issue #152 Error when loading dependencies from releases - Issue #168 Regression in preview fixed - Issue #189 Warnings: Resource not accessible by integration - Issue #190 PublishToEnvironment is not working with AL-Go-PTE@preview - Issue #186 AL-GO build fails for multi-project repository when there's nothing to build - When you have GitHub pages enabled, AL-Go for GitHub would try to publish to github_pages environment - Special characters wasn't supported in parameters to GitHub actions (Create New App etc.) ### Continuous Delivery - Added new GitHub Action "Deliver" to deliver build output to Storage or AppSource - Refactor CI/CD and Release workflows to use new deliver action - Custom delivery supported by creating scripts with the naming convention DeliverTo*.ps1 in the .github folder ### AppSource Apps - New workflow: Publish to AppSource - Continuous Delivery to AppSource validation supported ### Settings - New Repo setting: CICDPushBranches can be specified as an array of branches, which triggers a CI/CD workflow on commit. Default is main', release/\*, feature/\* - New Repo setting: CICDPullRequestBranches can be specified as an array of branches, which triggers a CI/CD workflow on pull request. Default is main - New Repo setting: CICDSchedule can specify a CRONTab on when you want to run CI/CD on a schedule. Note that this will disable Push and Pull Request triggers unless specified specifically using CICDPushBranches or CICDPullRequestBranches - New Repo setting: UpdateGitHubGoSystemFilesSchedule can specify a CRONTab on when you want to Update AL-Go System Files. Note that when running on a schedule, update AL-Go system files will perfom a direct commit and not create a pull request. - New project Setting: AppSourceContext should be a compressed json structure containing authContext for submitting to AppSource. The BcContainerHelperFunction New-ALGoAppSourceContext will help you create this structure. - New project Setting: AppSourceContinuousDelivery. Set this to true in enable continuous delivery for this project to AppSource. This requires AppSourceContext and AppSourceProductId to be set as well - New project Setting: AppSourceProductId should be set to the product Id of this project in AppSource - New project Setting: AppSourceMainAppFolder. If you have multiple appFolders, this is the folder name of the main app to submit to AppSource. ### All workflows - Support 2 folder levels projects (apps\w1, apps\dk etc.) - Better error messages for if an error occurs within an action - Special characters are now supported in secrets - Initial support for agents running inside containers on a host - Optimized workflows to have fewer jobs ### Update AL-Go System Files Workflow - workflow now displays the currently used template URL when selecting the Run Workflow action ### CI/CD workflow - Better detection of changed projects - appDependencyProbingPaths did not support multiple projects in the same repository for latestBuild dependencies - appDependencyProbingPaths with release=latestBuild only considered the last 30 artifacts - Use mutex around ReadSecrets to ensure that multiple agents on the same host doesn't clash - Add lfs when checking out files for CI/CD to support checking in dependencies - Continue on error with Deploy and Deliver ### CI/CD and Publish To New Environment - Base functionality for selecting a specific GitHub runner for an environment - Include dependencies artifacts when deploying (if generateDependencyArtifacts is true) ### localDevEnv.ps1 and cloudDevEnv.ps1 - Display clear error message if something goes wrong Co-authored-by: mazhelez <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Preview
Note that when using the preview version of AL-Go for GitHub, you need to Update your AL-Go system files, as soon as possible when told to do so.
Issues
Changes
Continuous Delivery
v2.0
Issues
Continuous Delivery
AppSource Apps
Settings
All workflows
Update AL-Go System Files Workflow
CI/CD workflow
CI/CD and Publish To New Environment
localDevEnv.ps1 and cloudDevEnv.ps1