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

Fix Crowdin Upload #1199

Merged

Conversation

lkiesow
Copy link
Member

@lkiesow lkiesow commented Oct 31, 2019

This patch fixes the Crowdin upload script which fails on non-release
branches right now:

return 0
.upload-crowdin.sh: line 12: return: can only `return' from a function or sourced script

Your pull request should…

  • have a concise title
  • close an accompanying issue if one exists
  • be against the correct branch (features can only go into develop)
  • include migration scripts and documentation, if appropriate
  • pass automated testing
  • have a clean commit history
  • have proper commit messages (title and body) for all commits
  • have appropriate tags applied

This patch fixes the Crowdin upload script which fails on non-release
branches right now:

    return 0
    .upload-crowdin.sh: line 12: return: can only `return' from a function or sourced script
@lkiesow lkiesow added elan e.V. Pull requests originating from elan e.V. bug labels Oct 31, 2019
@gregorydlogan gregorydlogan merged commit 08d2b2f into opencast:r/8.x Nov 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug elan e.V. Pull requests originating from elan e.V.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants