question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Remove 'squashed commits' requirement from CamperBot

See original GitHub issue

Now with Squash and merge, should we remove all references in the docs and Camperbot about squashing commits?

One reason for is, after reviewing a PR and requesting changes, we lose review comments made against a commit because it doesn’t exist any more. This makes checking that requested changes were made more of a memory game.

Issue Analytics

  • State:closed
  • Created 6 years ago
  • Reactions:6
  • Comments:9 (8 by maintainers)

github_iconTop GitHub Comments

2reactions
raisedadeadcommented, Jun 20, 2018

That is correct @Vimal-Raghubir You are more than welcome to take a look at https://github.com/freeCodeCamp/camper-gh-bot and wait for my progress there, and then on I would love your help on some of the user stories if you would be interested.

0reactions
Vimal-Raghubircommented, Jun 20, 2018

I would definitely be interested in helping out with the user stories. @raisedadead

Read more comments on GitHub >

github_iconTop Results From Across the Web

How to delete a sub commit after squashing commits?
Short answer. Use fixup instead of squash . It's like squash , but it will discard the commit messages for everything but the...
Read more >
Git Squash: How to Condense Your Commit History - CloudBees
In this post, we'll cover everything you need to know about Git Squash including what it does and best practices for using it....
Read more >
FreeCodeCamp/CoreTeam - Gitter
We've got camperbot patched to work over https and we are once again auto ... @benmcmahon100 please squash those commits real quick so...
Read more >
Squash and merge - GitLab Docs
Select or clear the Squash commits when merge request is accepted checkbox. ... To modify either the squash commit message or the merge...
Read more >
Public Lab: Blog
(If required) - Seek help from the mentors as well as fellow Outreachy Applicants ... to my notice that this functionality has been...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found