-
Notifications
You must be signed in to change notification settings - Fork 3
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
SA-3610 GHA Migration #64
Conversation
…loud/jcapi-powershell into SA-3610_GHA_Workflow_Migrations
@@ -291,7 +290,7 @@ ForEach ($SDK In $SDKName) | |||
# Remove auto generated .gitignore files | |||
If ($ModifyGitIgnore) | |||
{ | |||
$GitIgnoreFiles = Get-ChildItem -Path:($OutputFullPath) -Recurse -File | Where-Object { $_.Extension -eq '.gitignore' } | |||
$GitIgnoreFiles = Get-ChildItem -Path:($OutputFullPath) -Recurse -File -Hidden | Where-Object { $_.Extension -eq '.gitignore' } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
required to update .gitignore files
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not seeing anything that I would change. Everything looks great, nice job! Let's see how a deployment attempt goes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Workflows and tests looks good, only change I'm thinking of is removing the .circleCi folder since we do not need it anymore
Issues
What does this solve?
Replace CircleCI with GHA. Release
Start-JCSdkCommand
function within the V1 module.Is there anything particularly tricky?
Not much has changed, a few things have been removed in favor of faster tests.
How should this be tested?
Team to discuss and show how module release steps have been tested.
Screenshots