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

Issue 170 #171

Closed
wants to merge 4 commits into from
Closed

Issue 170 #171

wants to merge 4 commits into from

Conversation

n3pjk
Copy link

@n3pjk n3pjk commented Aug 1, 2023

Besides the custom themes and plugins, I noticed that there's no check if the user is managed locally or not. Our desktops are joined to a domain and managed with PBIS. I put the defaults into a default dictionary to simplifies conditionals. I cleaned up the jinja scripts in the template and moved the user loop up to main, simplifying the install code.

@CLAassistant
Copy link

CLAassistant commented Aug 1, 2023

CLA assistant check
All committers have signed the CLA.

@n3pjk
Copy link
Author

n3pjk commented Aug 2, 2023

I've also made the .zshrc jinja template able to be overridden.

@freemanjp
Copy link
Member

Hi @n3pjk. I appreciate the PR (you clearly put a lot of work into it), but unfortunately, I've decided to depreciate this Ansible role. It's been years since I've used this role myself and I no longer have time to maintain it.

I've moved on to using a Zsh Plugin manager (zimfw) and a custom dotfile repo (https://code.visualstudio.com/docs/devcontainers/containers#_personalizing-with-dotfile-repositories).

You're welcome to fork this role, but the tests are broken due to incompatible Python dependency changes. I've been moving my roles to use tox for testing (in place of my Molecule Wrapper script) and freezing all the dependency versions, but this role hasn't been updated.

@n3pjk
Copy link
Author

n3pjk commented Aug 7, 2023 via email

@freemanjp
Copy link
Member

@n3pjk, unfortunately, given this Ansible role is under my company gantsign namespace, I can't make someone else maintainer (it'd expose me to legal risk if there were any issues). There's also no way of transferring an Ansible Galaxy role to another namespace. I would suggest that you fork the role (it is MIT licensed), and update the README.md to make it clear it's no longer a role maintained by me or GantSign. I could then update the README.md in my repo to link to your fork.

@freemanjp freemanjp closed this Sep 17, 2023
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.

3 participants