From 62463d2c3e63ebd25f9f54e5bb0e950f4b1a0e93 Mon Sep 17 00:00:00 2001 From: Christoph Date: Thu, 26 Oct 2017 23:35:55 +0200 Subject: [PATCH] Remove notes about the signature (#4015) * Remove notes about the signature It's handled by a bot now :tada: * Remove check from travis * Remove check from appveyor --- .appveyor/check.bat | 30 ------------------------- .github/CONTRIBUTING.md | 49 +---------------------------------------- .travis.yml | 2 -- .travis/check.sh | 25 --------------------- appveyor.yml | 3 --- 5 files changed, 1 insertion(+), 108 deletions(-) delete mode 100644 .appveyor/check.bat delete mode 100755 .travis/check.sh diff --git a/.appveyor/check.bat b/.appveyor/check.bat deleted file mode 100644 index 54c1d3df7..000000000 --- a/.appveyor/check.bat +++ /dev/null @@ -1,30 +0,0 @@ -@echo off - -call:checkCommitMessage -GOTO:EOF - -:checkCommitMessage - call:logInfo "Commit message: %APPVEYOR_REPO_COMMIT_MESSAGE%" - call:logInfo "Is pull request: %APPVEYOR_PULL_REQUEST_NUMBER%" - - if not "%APPVEYOR_PULL_REQUEST_NUMBER%" == "" ( - ECHO "%APPVEYOR_REPO_COMMIT_MESSAGE_EXTENDED%" | FINDSTR /C:"Signed-off-by: " >nul & IF ERRORLEVEL 1 ( - call:logError "The commit message does not contain the signature!" - call:logError "More information: https://github.com/telegramdesktop/tdesktop/blob/master/.github/CONTRIBUTING.md#sign-your-work" - exit 1 - ) else ( - call:logInfo "Commit message contains signature" - - :: Reset error level - verify >nul - ) - ) -GOTO:EOF - -:logInfo - echo [INFO] %~1 -GOTO:EOF - -:logError - echo [ERROR] %~1 -GOTO:EOF diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md index 37c76d9d3..cef07b094 100644 --- a/.github/CONTRIBUTING.md +++ b/.github/CONTRIBUTING.md @@ -5,8 +5,6 @@ This document describes how you can contribute to Telegram Desktop. Please read **Table of Contents** * [What contributions are accepted](#what-contributions-are-accepted) -* [Sign your work](#sign-your-work) - * [Change commit message of a pushed commit](#change-commit-message-of-a-pushed-commit) * [Build instructions](#build-instructions) * [Pull upstream changes into your fork regularly](#pull-upstream-changes-into-your-fork-regularly) * [How to get your pull request accepted](#how-to-get-your-pull-request-accepted) @@ -28,49 +26,6 @@ Unfortunately we **do not merge** any pull requests that have new feature implem Telegram Desktop is not a standalone application but a part of [Telegram project][telegram], so all the decisions about the features, languages, user experience, user interface and the design are made inside Telegram team, often according to some roadmap which is not public. -## Sign your work - -For contributions to be accepted they should be granted into the public domain. This will solve the issue if Telegram team needs to use full Telegram Desktop source code with some different license. - -The sign-off is a simple line at the end of the explanation for the patch. Your signature certifies that you wrote the patch and you have the right to put it in the public domain. The rules are pretty simple: if you can certify the below: - -``` -Telegram Desktop Developer Certificate of Origin - -By making a contribution to this project, I certify that: - -(a) The contribution was created in whole by me or is based upon - previous work that, to the best of my knowledge, is in the - public domain and I have the right to put it in the public domain. - -(d) I understand and agree that this project and the contribution are - public and that a record of the contribution (including all - metadata and personal information I submit with it, including my - sign-off) is maintained indefinitely and may be redistributed. - -(e) I am granting this work into the public domain. -``` - -Then you just add a line to every **git commit message** that states: - - Signed-off-by: Random J Developer (github: rndjdev_github) - -Replacing Random Developer’s details with your name, email address and GitHub username. - -### Change commit message of a pushed commit - -If you already pushed a commit and forgot to add the signature to the commit message, follow these steps to change the message of the commit: - -1. Open `Git Bash` (or `Git Shell`) -2. Enter following command to change the commit message of the most recent commit: `git commit --amend` -3. Press i to get into Insert-mode -4. Change the commit message (and add the [signature](#sign-your-work) at the and) -5. After editing the message, press ESC to get out of the Insert-mode -6. Write `:wq` and press Enter to save the new message or write `:q!` to discard your changes -7. Enter `git push --force` to push the commit with the new commit message to the remote repository - -For more info, see [GitHub Help][help_change_commit_message]. - ## Build instructions See the [README.md][build_instructions] for details on the various build @@ -118,7 +73,7 @@ If you already have multiple commits, you can add the commits together (squash t 1. Open `Git Bash` (or `Git Shell`) 2. Enter following command to squash the recent {N} commits: `git reset --soft HEAD~{N} && git commit` (replace `{N}` with the number of commits you want to squash) 3. Press i to get into Insert-mode -4. Enter the commit message of the new commit (and add the [signature](#sign-your-work) at the end) +4. Enter the commit message of the new commit 5. After adding the message, press ESC to get out of the Insert-mode 6. Write `:wq` and press Enter to save the new message or write `:q!` to discard your changes 7. Enter `git push --force` to push the new commit to the remote repository @@ -150,8 +105,6 @@ Before you submit a pull request, please test your changes. Verify that Telegram For example: `Fix #545` -* Don't forget to [sign your patch](#sign-your-work) to put it in the public domain! - [//]: # (LINKS) [telegram]: https://telegram.org/ [help_fork_repo]: https://help.github.com/articles/fork-a-repo/ diff --git a/.travis.yml b/.travis.yml index 03710c3da..13ba3348c 100644 --- a/.travis.yml +++ b/.travis.yml @@ -58,8 +58,6 @@ addons: - yasm before_install: - - "export TRAVIS_COMMIT_MSG=\"$(git log --format=%B --no-merges -n 1)\"" - - .travis/check.sh - export CXX="g++-6" CC="gcc-6" - sudo update-alternatives --install /usr/bin/gcc gcc /usr/bin/gcc-6 60 --slave /usr/bin/g++ g++ /usr/bin/g++-6 - sudo update-alternatives --config gcc diff --git a/.travis/check.sh b/.travis/check.sh deleted file mode 100755 index 3985ac22e..000000000 --- a/.travis/check.sh +++ /dev/null @@ -1,25 +0,0 @@ -#!/bin/bash -# Checks if the commit message contains the signature - -run() { - checkCommitMessage -} - -checkCommitMessage() { - info_msg "Commit message: ${TRAVIS_COMMIT_MSG}"; - info_msg "Is pull request: ${TRAVIS_PULL_REQUEST}"; - - if [[ $TRAVIS_PULL_REQUEST != "false" ]];then - if [[ $TRAVIS_COMMIT_MSG != *"Signed-off-by: "* ]];then - error_msg "The commit message does not contain the signature!" - error_msg "More information: https://github.com/telegramdesktop/tdesktop/blob/master/.github/CONTRIBUTING.md#sign-your-work" - exit 1 - else - success_msg "Commit message contains signature" - fi - fi -} - -source ./.travis/common.sh - -run diff --git a/appveyor.yml b/appveyor.yml index d436d5683..da1d2df46 100644 --- a/appveyor.yml +++ b/appveyor.yml @@ -14,9 +14,6 @@ environment: matrix: fast_finish: true -install: - - .\.appveyor\check.bat - before_build: - .\.appveyor\install.bat