-
Notifications
You must be signed in to change notification settings - Fork 41
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
Add more support for Azure DevOps #204
Conversation
bc5bd08
to
b290447
Compare
a460103
to
2ac27ec
Compare
Finished porting the code changes, but still need to fix some tests. |
2ac27ec
to
f45003b
Compare
f45003b
to
4fee22f
Compare
(We need to add some better command line testing around the positional arguments.) This doesn't work with this PR:
I would guess it's because we're now setting the branch to an empty string and the Updater thinks an empty string branch name is ok. To fix it, I would change the Source struct to not have a pointer to |
Thanks for the sanity check. I made |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work 👍
Adds the following features:
--api-url
flag which gets passed through as theDEPENDABOT_API_URL
environment variable for the updater containerhttp://host.docker.internal
if one isn't providedLOCAL_AZURE_ACCESS_TOKEN
environment variableDEPENDABOT_JOB_ID
environment variable to the updater containercli
job ID if the environment isn't setTODO:
Additional support for setting input and job parameters from environment variables