Skip to content

Conversation

@pedmon
Copy link
Contributor

@pedmon pedmon commented May 2, 2023

We have a secure environment that requires proxy to get out for dnf repos. This adds the option to set a proxy for the OnDemand repo.

@treydock
Copy link
Collaborator

treydock commented May 3, 2023

@pedmon I left inline suggestion. Generally you want default values to be undef if the parameter is optional.

@treydock treydock added the enhancement New feature or request label May 3, 2023
@pedmon
Copy link
Contributor Author

pedmon commented May 3, 2023

Cool. I've accepted that suggestion.

@treydock
Copy link
Collaborator

treydock commented May 3, 2023

@pedmon Can you rebase against latest master branch? I pushed some changes that should resolve failing tests.

@pedmon
Copy link
Contributor Author

pedmon commented May 3, 2023

Okay, rebased

@treydock treydock merged commit 08d092d into OSC:master May 4, 2023
@treydock
Copy link
Collaborator

treydock commented May 4, 2023

This is being released with v4.0.0 once Github Actions complete

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants