diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md index 4dd9fc868..7ada159cb 100644 --- a/.github/CONTRIBUTING.md +++ b/.github/CONTRIBUTING.md @@ -68,8 +68,28 @@ git checkout -b your-branch-name - If your checks have passed, your PR will be assigned a reviewer who will review your code and provide comments. Please address each review comment by pushing new commits to the same branch (the PR will automatically update, so you don’t need to submit a new one). Once you are done, comment below each review comment marking it as “Done”. Feel free to use the thread to have a discussion about comments that you don’t understand completely or don’t agree with. - - Once all comments are addressed, the reviewer will give an LGTM (‘looks good to me’) and merge the PR. -4. Rules for great commit messages: + - Once all comments are addressed, the maintainer will approve the PR. + +4. Once you get reviewed by a mentor and done with all the required changes, squash all the commits: + +``` +git checkout +git rebase -i HEAD~N (N is the number of commits to be squashed) +``` + +Then a screen will appear with all N commits having "pick" written in front of every commit.Change pick to s for the last N-1 commits and let it be pick for the first one. + +Press esc button and type ":wq" to save the change and close the screen. Now a new screen will appear asking you to change commit message. Change it accordingly and save it. + +``` +git push origin --force +``` + +For further query regarding rebasing, visit https://github.com/todotxt/todo.txt-android/wiki/Squash-All-Commits-Related-to-a-Single-Issue-into-a-Single-Commit + +Once rebasing is done, the reviewer will approve and merge the PR. + +5. Rules for great commit messages: - Write your commit message in the imperative: "Fix bug" and not "Fixed bug" or "Fixes bug." @@ -85,7 +105,7 @@ git checkout -b your-branch-name - Describe why a change is being made. -5. Commit message is important as it should answers/should do the following: +6. Commit message is important as it should answers/should do the following: - How does it address the issue? @@ -95,7 +115,7 @@ git checkout -b your-branch-name - Provide concise explaination to the user about limitations of the current code, like a bug or an issue -6. Examples of a great commit message: +7. Examples of a great commit message: - Add CPU arch filter scheduler support diff --git a/.travis.yml b/.travis.yml index 2f9bbbec8..17b75afd1 100644 --- a/.travis.yml +++ b/.travis.yml @@ -2,6 +2,9 @@ sudo: required dist: trusty language: node_js addons: + apt: + packages: + - dpkg chrome: stable node_js: diff --git a/src/app/components/challenge/challengesettings/challengesettings.component.ts b/src/app/components/challenge/challengesettings/challengesettings.component.ts index f28248449..d358adbec 100644 --- a/src/app/components/challenge/challengesettings/challengesettings.component.ts +++ b/src/app/components/challenge/challengesettings/challengesettings.component.ts @@ -105,11 +105,14 @@ export class ChallengesettingsComponent implements OnInit { */ remove(email): void { const SELF = this; - const index = this.bannedEmailIds.indexOf(email); + const index = SELF.bannedEmailIds.indexOf(email); if (index >= 0) { - this.bannedEmailIds.splice(index, 1); + SELF.bannedEmailIds.splice(index, 1); } + + // updating the banned Email Ids list + SELF.updateBannedEmailList(); } validateEmail(email) {