Skip to content
This repository was archived by the owner on Jun 30, 2021. It is now read-only.

Conversation

@lerua83
Copy link
Contributor

@lerua83 lerua83 commented Sep 25, 2017

Fixing some typo errors on VNC ports

Fixing some typo errors on VNC ports
@CLAassistant
Copy link

CLAassistant commented Sep 25, 2017

CLA assistant check
All committers have signed the CLA.

@elgalu
Copy link
Owner

elgalu commented Sep 26, 2017

The build failure is unrelated now:

die 'Need env var DOCKER_USERNAME to push to docker'

I have to sort out the build for non-trusted PRs but I'm eager to merge, thanks!!

@elgalu
Copy link
Owner

elgalu commented Sep 27, 2017

@lerua83 can you merge the updates I put in the master branch so the build triggers again, hopefully I fixed the PRs problem.

@lerua83 lerua83 closed this Oct 2, 2017
@lerua83 lerua83 reopened this Oct 2, 2017
@lerua83
Copy link
Contributor Author

lerua83 commented Oct 2, 2017

Hi,

I'm trying to make a merge of the pull request but unfortunately I don't see the option. Instead of that I see the message Only those with write access to this repository can merge pull requests.
Am I doing something wrong or Have I missed some option?

Thnaks in advance,
Álvaro León

@elgalu elgalu merged commit 02ab85f into elgalu:master Oct 2, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants