Skip to content
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

Update 2-setup-azure-environment.yml #49

Merged
merged 10 commits into from Jun 5, 2023
Merged

Conversation

shearpaladin
Copy link
Contributor

@shearpaladin shearpaladin commented Jun 4, 2023

Updated Lines:
86: 'env' -> 'with'
92: 'env' -> 'with'
98: 'env' -> 'with'

GitHub Actions kept failing on this step

Updated Line:
110: ./.github/script/initialize-repository.sh

Using the correct script path because pull-main.sh does not exist

Summary

#47

Changes

Fixes Github Actions Step 2 Failure.
Changed 'env' - > 'with'
Updated file path to correct script

Task list

  • For workflow changes, I have verified the Actions workflows function as expected.
  • For content changes, I have reviewed the style guide.

Updated Lines:
86: 'env' -> 'with'
92: 'env' -> 'with'
98: 'env' -> 'with'

GitHub Actions kept failing on this step

Updated Line:
110: ./.github/script/initialize-repository.sh

Using the correct script path because pull-main.sh does not exist
@shearpaladin
Copy link
Contributor Author

@heiskr
I've created a pull request to solve the issues found in Step 2.

(This is my first PR on something public so I'd like some feedback too 👍 )

@shearpaladin
Copy link
Contributor Author

shearpaladin commented Jun 4, 2023

Update: this issue is on Multiple steps.. it seems i see it on step 3.. we have to change all the paths.

I believe it might be better to rename that script to pull-main.sh since it's in all the steps

Changed back to pull-main.sh
Changed file name to match path
@shearpaladin
Copy link
Contributor Author

Changed the file name to pull-main.sh

@heiskr
Copy link
Contributor

heiskr commented Jun 5, 2023

Thank you for the pull request. Can we keep the env->with for this one, and a separate one to address scripts?

Thanks ✨

@heiskr heiskr requested a review from a team June 5, 2023 15:22
@shearpaladin
Copy link
Contributor Author

shearpaladin commented Jun 5, 2023

@heiskr

Thank you for the pull request. Can we keep the env->with for this one, and a separate one to address scripts?

Thanks ✨

Yes for sure!

Reverted back to pull-main.sh
@shearpaladin
Copy link
Contributor Author

shearpaladin commented Jun 5, 2023

Okay, it should be changed with only 'with'.

I'll create a separate PR for scripts

#50

@heiskr heiskr merged commit ef3e35d into skills:main Jun 5, 2023
@heiskr
Copy link
Contributor

heiskr commented Jun 5, 2023

Thank you :)

@shearpaladin shearpaladin deleted the step-2-fix branch June 5, 2023 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants