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

Forced commit on "Refresh Model" #224

Open
timolaine opened this issue Oct 17, 2024 · 5 comments
Open

Forced commit on "Refresh Model" #224

timolaine opened this issue Oct 17, 2024 · 5 comments

Comments

@timolaine
Copy link

I am not sure this is actually a bug, but it is something that I do not completely understand. I am not sure what "Refresh Model" does.

I thought that "Refresh Model" just does a git pull, but apparently there is more to it. To me, it seems like some (all?) changes to the model actually require me to commit (not to push, just to commit) my changes as I do a Refresh.

Version of Archi, Operating System

Windows

Version of this plug-in

0.9.2

Any other plug-ins installed?

jArchi

Expected Behaviour

  1. Refresh Model.
  2. Make a small change to a model.
  3. Refresh Model
  4. Continue modelling without having to commit.

Actual Behaviour

  1. Refresh Model.
  2. Make a small change to a model.
  3. Refresh Model and be forced to commit along with the Refresh.
    or
  4. Commit
  5. Refresh Model

Steps to Reproduce the Behaviour

  1. Refresh Model.
  2. Make a small change to a model.
  3. Refresh Model
@Phillipus
Copy link
Member

Hi, yes this is intended behaviour. Pending changes need to be committed first in order to "Refresh" which is a git pull and merge.

@timolaine
Copy link
Author

Thank you @Phillipus! I think this behaviour should be documented somewhere, at least in some condensed form. I had always thought that if you had no conflicts, there was no need to commit on Refresh. Perhaps the problem is that I don't understand Git itself that well, but I am quite sure that is common to many coArchi users.

@Phillipus
Copy link
Member

I think this behaviour should be documented somewhere

If you think the wiki could be improved you are free to make a suggestion and/or edit it if you like.

@timolaine
Copy link
Author

Thank you, I updated the documentation. I hope my addition is correct.

@Phillipus
Copy link
Member

Thank you, I updated the documentation. I hope my addition is correct.

Thanks!

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

2 participants