Skip to content
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

403 fix #170

Merged
merged 4 commits into from Nov 13, 2015
Merged

403 fix #170

merged 4 commits into from Nov 13, 2015

Conversation

drzowie
Copy link
Contributor

@drzowie drzowie commented Nov 13, 2015

fixes bug 403

@wchristian wchristian closed this Nov 13, 2015
@wchristian wchristian deleted the 403-fix branch November 13, 2015 05:45
@zmughal zmughal restored the 403-fix branch November 13, 2015 06:14
@wchristian wchristian deleted the 403-fix branch November 13, 2015 06:15
@zmughal
Copy link
Member

zmughal commented Nov 13, 2015

If you try to push to the master on GitHub to close the PR, it will be reverted automatically. You have to push to the branch on SourceForge. This is a "feature" of @wchristian's sync script.

Also, do not do a merge before the CI jobs are finished, otherwise they will fail because it will not know where to pull from. :-)

@zmughal
Copy link
Member

zmughal commented Nov 13, 2015

I am going to rebuild the last commit on the CI.

@zmughal zmughal restored the 403-fix branch November 13, 2015 06:17
@zmughal zmughal reopened this Nov 13, 2015
@zmughal
Copy link
Member

zmughal commented Nov 13, 2015

This has passed on all CI systems. Going to merge it in.

@zmughal zmughal merged commit 8db4227 into master Nov 13, 2015
@devel-chm
Copy link
Member

If the topic branch has been merged into master, can it be deleted now? Hard to track what is in the sf.net repo with dangling branches. :-)

@zmughal
Copy link
Member

zmughal commented Nov 14, 2015

Aye, I shall do that now.

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.

None yet

4 participants