autopkg / MLBZ521-recipes

AutoPkg Recipes
MIT License
14 stars 22 forks source link

Updated Download Variable Names #70

Closed SeanHansell closed 1 year ago

SeanHansell commented 1 year ago

Fix for #69

Tested and new variable names do not conflict with JamfUpload variables. Unfortunately users will need to update their overrides, butI don't think there's any way around it.

SeanHansell commented 1 year ago

I renamed all the variables just for consistency's sake, even though only two of them are causing an issue now.

vao-brex commented 1 year ago

+1

MLBZ521 commented 1 year ago

@SeanHansell I'm trying to remove the .pyc file and pycache directory in this PR, but both from the web and local GitHub app, I'm getting errors. For some reason, it won't let me as it says your master branch is protected -- even though it doesn't appear so when I looked at your fork.

I've submitted a PR on your fork...hopefully that'll work.

SeanHansell commented 1 year ago

@MLBZ521 Merged! This should be good to go!

MLBZ521 commented 1 year ago

Thanks!