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

Changes count not reseting #147

Closed
aonez opened this issue Oct 26, 2017 · 21 comments
Closed

Changes count not reseting #147

aonez opened this issue Oct 26, 2017 · 21 comments
Milestone

Comments

@aonez
Copy link

aonez commented Oct 26, 2017

Reported via Twitter.

The counter increments correctly, but It does not drop when a commit is done. Can't see any log relating to this issue.

  • On a machine I got this issue and after updating and rolling back it's no more present.
  • On my primary machine I've created a test repository and now I have no counter at all:

screen shot 2017-10-26 at 12 46 49

Reproduced in OS X 10.11.6 and macOS 10.13. From 1.0.57.6 to 1.0.58.2.

@DanPristupov
Copy link
Contributor

A lot of people have reported this issue, but I still can't reproduce it. I'll continue investigating the problem.

Please share if you find more details.

@clounie
Copy link

clounie commented Oct 29, 2017

I don't have a hard repro at this point, but from my experience it generally happens when I'm using the git CLI + Fork at the same time.

@aonez
Copy link
Author

aonez commented Oct 30, 2017

@clounie I only work with git CLI when Fork fails on a rebase or something involving too much changes. Usually I only use Fork, and the count fails to reset.

@bwl21
Copy link

bwl21 commented Jan 2, 2018

I did a git gc on the command line, maybe this might help. Anyhow the changes counter is not really reliable. I even would prefer if the uncommitted changes would be visible the commit history (see my comment on #107)

@DanPristupov
Copy link
Contributor

I'm finally back from the holidays and investigating this bug now.

@bwl21 I recently added additional logging related to this problem. You can really help me to fix it if you send me the log file which is located at ~/Library/Caches/Fork/fork.log to support@git-fork.com. Thank you in advance.

@bwl21
Copy link

bwl21 commented Mar 19, 2018

As a pity, the issue is still there in 1.0.65.1

@aonez
Copy link
Author

aonez commented Mar 19, 2018

send me the log file which is located at ~/Library/Caches/Fork/fork.log to support@git-fork.com.

You still need those? I can send you mine too.

I'm now somewhat used to don't care about the count number, since it is not reliable. But Fork still rocks hard, little issue really.

@aonez
Copy link
Author

aonez commented Jul 30, 2018

Working again for me in the last hot-fix update 1.0.69.4 😎

@DanPristupov
Copy link
Contributor

Can we close that now (in 1.0.72)?

@DanPristupov DanPristupov added this to the 1.0.72 milestone Dec 1, 2018
@aonez
Copy link
Author

aonez commented Dec 2, 2018

I can't even reproduce this with 1.0.71.2. So for my part you can close this. Hopping to see the day when we can support this project 💰

@bwl21
Copy link

bwl21 commented Dec 2, 2018

as a pity I still have the issue in 1.0.72.1. In the following screenshot you see, there is a chage, but
in the "changes"

screenshot_1271

@aonez
Copy link
Author

aonez commented Dec 2, 2018

@bwl21 what OS do you use? Also, the repository is local/remote and on what hosting if the later?

@aonez
Copy link
Author

aonez commented Dec 2, 2018

By the way the buttons on your screenshot look funny.

screen shot 2018-12-02 at 11 52 51

Do you use the old toolbar style or it is imposed for using an old OS?

@bwl21
Copy link

bwl21 commented Dec 2, 2018

I am on el capitan. repository is hosted github. But not all branches are pushed there.

@aonez
Copy link
Author

aonez commented Dec 2, 2018

Thanks @bwl21! @DanPristupov I was also working on an old Mac Mini with Mac OS X 10.11 El Capitan when reported this one. Now I can't reproduce but my minimum OS is 10.13. Will try to reproduce on a VM. Nah, forget it:

Reproduced in OS X 10.11.6 and macOS 10.13. From 1.0.57.6 to 1.0.58.2.

Forgot this line of the first report. So the OS should no be the reason.

@bwl21
Copy link

bwl21 commented Dec 2, 2018

Do you use the old toolbar style or it is imposed for using an old OS?

i did not change the toolbar setting up to now, so it is the old style

@bwl21
Copy link

bwl21 commented Dec 2, 2018

Forgot this line of the first report. So the OS should no be the reason.

I tried with https://github.com/bwl21/react-jspdf-example, a very small repository. It still happens that the Counter is not updated.

screenshot_1272

@DanPristupov
Copy link
Contributor

DanPristupov commented Dec 2, 2018

@aonez, @bwl21
Can I create for you a special Fork build with additional logging which will help to find the bug?

@bwl21
Copy link

bwl21 commented Dec 2, 2018

sure. can it be installed in parallel?

@aonez
Copy link
Author

aonez commented Dec 3, 2018

If I can reproduce this again (not the case now) I'll for sure use the build with additional logging.

@DanPristupov
Copy link
Contributor

OK, I'm investigating this problem right now.

sure. can it be installed in parallel?

Yes, it won't require installation, you can run it right from the DMG file.

@aonez @aonez thank you! Please contact me by email (support@forkapp.io) so I could send you the build.

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

No branches or pull requests

4 participants