Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Uninstall ansible if version not same (needed to upgrade to 2.10)#11

Merged
seanwang-okta merged 1 commit intostrapsh:masterfrom
seanwang-okta:sw-ansible
Mar 2, 2021
Merged

Uninstall ansible if version not same (needed to upgrade to 2.10)#11
seanwang-okta merged 1 commit intostrapsh:masterfrom
seanwang-okta:sw-ansible

Conversation

@seanwang-okta
Copy link
Collaborator

Need to update Ansible version to 2.10.5 or greater due to homebrew cask changes which is causing issues: ansible-collections/community.general#1524

In order to update Ansible from 2.9.10 to 2.10, need to uninstall first, then reinstall: https://docs.ansible.com/ansible/latest/installation_guide/intro_installation.html#installing-ansible-on-macos

Copy link
Contributor

@lhazlewood lhazlewood left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@seanwang-okta
Copy link
Collaborator Author

hey @lhazlewood can you merge this for me? i don't have write access.

@seanwang-okta seanwang-okta requested a review from lhazlewood March 2, 2021 20:25
@lhazlewood
Copy link
Contributor

@seanwang-okta just invited you as a team member - please ensure you can merge :)

@seanwang-okta
Copy link
Collaborator Author

@lhazlewood i accepted the invitation but still unable to merge. Do I need to be granted explicit write access? thanks.

@seanwang-okta
Copy link
Collaborator Author

Screen Shot 2021-03-02 at 5 00 45 PM

@lhazlewood
Copy link
Contributor

Maybe it was just that someone needed to approve the review?

@seanwang-okta
Copy link
Collaborator Author

hmmm still doesn't work. i can see the approval but it still says "Only those with write access to this repository can merge pull requests."

@lhazlewood
Copy link
Contributor

Just added write access - sorry!

@seanwang-okta seanwang-okta merged commit 0d485fd into strapsh:master Mar 2, 2021
@seanwang-okta
Copy link
Collaborator Author

awesome that worked, thanks @lhazlewood :)

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.

2 participants