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

Suggest git stash around upgrade #1262

Closed
wants to merge 1 commit into from
Closed

Suggest git stash around upgrade #1262

wants to merge 1 commit into from

Conversation

mthmulders
Copy link
Contributor

Naar aanleiding van #1259 en 693cff7: handmatige wijzigingen kunnen met git stash en git stash pop geback-upped en teruggezet worden.

@mthmulders mthmulders changed the base branch from v4 to development January 17, 2021 14:32
@codecov-io
Copy link

Codecov Report

Merging #1262 (10e4821) into development (5eefa66) will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff              @@
##           development     #1262   +/-   ##
=============================================
  Coverage       100.00%   100.00%           
=============================================
  Files              121       121           
  Lines             3947      3947           
  Branches           343       343           
=============================================
  Hits              3947      3947           

@dennissiemensma
Copy link
Member

Bedankt voor je suggestie! Ik kan me als programmeur goed vinden in je tips voor de gebruiker, alleen zou ik graag het graag willen richten op het meerendeel van de gebruikers, die vermoedelijk uberhaupt niet weten hoe git werkt of wat het doet.

Ik denk dat ik eerst aankijk of er zich later uberhaupt gebruikers melden, die zien dat ze openstaande wijzigingen hebben en ze willen bewaren. Zelf zit ik er namelijk niet op te wachten op uitgebreide ondersteuning te (moeten) geven op wanneer ze zelf gaan mergen of conflicten krijgen. Dus ik houd je PR voor later achter de hand!

@dennissiemensma
Copy link
Member

Voor nu sluit ik deze. Mochten veel gebruikers toch tegen bovenstaande aanlopen, dan zal ik het herzien!

@dennissiemensma dennissiemensma added this to the Other milestone Jan 17, 2021
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

Successfully merging this pull request may close these issues.

3 participants