-
Notifications
You must be signed in to change notification settings - Fork 132
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 build command extension (details, delete, keep, queue with params and priority), add build agents and pools commands, add build template commands, add build definition cmmands (include nchabra definition commands) #85
base: master
Are you sure you want to change the base?
Conversation
can someone please comment on the ability to merge my changes? |
@bryanmacfarlane Can you take a look at this? |
@bryanmacfarlane Please have a look |
Hi Guys ... i managed to do all refactoring and test my version (all features), my team is starting a VSTS Proof of concept, we are expected to add more features and i really want to merge our already in use changes to the master fork. |
I wonder if I can summon @msftclas ... :) |
Oops I overlooked the signature part. |
i added build details extended show - i think this might com in handy for everyone.
i added build/delete.ts (right now not working - still can figure out why) - feel free to exclude.
added debug message in login.ts to show the actual response in the login (error or message), this helped me debug the login issues i had.
also added build queue with priority argument and parameters argument (path to simple JSON with build process parameters) - this is very useful for automation of build chain.