Skip to content

Use Start-Process to start pageant.exe#254

Merged
johnhaley81 merged 1 commit intonodegit:appveyor-agentfrom
FeodorFitsner:appveyor-agent
Oct 25, 2014
Merged

Use Start-Process to start pageant.exe#254
johnhaley81 merged 1 commit intonodegit:appveyor-agentfrom
FeodorFitsner:appveyor-agent

Conversation

@FeodorFitsner
Copy link
Copy Markdown
Contributor

@johnhaley81
Copy link
Copy Markdown
Collaborator

😄

@maxkorp
Copy link
Copy Markdown
Collaborator

maxkorp commented Oct 24, 2014

Awesome!

@johnhaley81
Copy link
Copy Markdown
Collaborator

@FeodorFitsner related note since we're waiting now on AppVeyor to run these tests before the merge. Is there a way to speed up these tests or cancel ones in the queue? Or maybe move one to the top of the queue?

@FeodorFitsner
Copy link
Copy Markdown
Contributor Author

Hm, I see what you mean. Very active projects have lot of commits thus overwhelming slow queue. There is only one concurrent job is allowed for free accounts.

Of course, you can just cancel those pending builds.

I recall we were discussing this issue on another OSS project. There was an idea of "rolling" builds, so the system waits until the build is completed and then queues another build of all check-ins that have not yet been built. This could be done per-branch I think. It seems we should get back to this idea again.

@johnhaley81
Copy link
Copy Markdown
Collaborator

That would be awesome. When you're making a lot of commits it sucks to have to wait 2 hours to see if something broke.

johnhaley81 added a commit that referenced this pull request Oct 25, 2014
Use Start-Process to start pageant.exe
@johnhaley81 johnhaley81 merged commit db2ecdd into nodegit:appveyor-agent Oct 25, 2014
@johnhaley81
Copy link
Copy Markdown
Collaborator

Thanks for the PR!

@FeodorFitsner
Copy link
Copy Markdown
Contributor Author

No prob! Glad AV works for your project.

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