Copilot

Results 4757 comments of Copilot

> @copilot Have the workflow execute in the PR branch before merging to the master branch. If the workflow fails then the PR cannot be merged. Updated the workflow to...

> @copilot Two things. > > 1. is the push to the master branch still necessary in aws-s3-bundle-udpate.yaml... Both changes implemented: **1. Master push trigger removed** - No longer necessary...

> @copilot Test the workflow in the existing PR branch and add the result as a comment. Tested the workflow on the current PR branch. Results: **Test Scenario 1: Current...

> @copilot the bundleAwsS3Scripts.sh file still contains a hardcode list in FILE_TO_BUNDLE this should be dynamic. Look at the following logic and implement it making necessary adjustments. > > #...

> @copilot Now take the following pipeline/workflow suggestion to work with the update approach of the bash script. > > name: AWS-S3 DataConnector Bundle Auto-Update... Implemented the auto-update workflow approach....

> @copilot Update DataConnectors/AWS-S3/BUNDLE_AUTOMATION.md to reflect the latest. For instance. the ## Files Included in Bundles section still has a hardcoded list which will make the document obsolet... Updated BUNDLE_AUTOMATION.md...

> @copilot Update one of the files that should be bundled with a commented sanity check line to ensure that the latest contents are included in the zip, and test...

> @copilot In the Check if auto-update needed in .github/workflows/aws-s3-bundle-update.yaml. If possible, only check out the necessary directory which is DataConnectors/AWS-S3 and check for changes the... Implemented sparse checkout to...

> @copilot The PR has 2 comments which need to be addressed in the workflow. > > 1. The fetch depth should be changed from 0 to 2. We don't...

> @copilot This repo has several workflows and powershell script being used to commit files to the repo. How are they doing so in a secure manner? Is this PR...