Skip to content
This repository has been archived by the owner on Feb 3, 2021. It is now read-only.

Fix typo in git_status section (deleted != unmerged) #159

Merged
merged 1 commit into from
Feb 17, 2019

Conversation

willnevillain
Copy link
Contributor

@willnevillain willnevillain commented Feb 17, 2019

In the git status section, when one has unmerged files, the character for deleted files will appear in the git status section instead.

Description

Changed 'deleted' to 'unmerged' in the 'if unmerged' block

Motivation and Context

Show correct character in git status section when there are unmerged files.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Screenshots (if appropriate):

Before:
image
After:
image

How Has This Been Tested?

Created git repo with unmerged files (merge conflict), saw it displaying incorrect character, made the variable change, started new fish in the same git repo dir and saw it fixed.

  • I have tested using MacOS
  • I have tested using Linux

Checklist:

  • I have checked that no other PR duplicates mine
  • My code follows the code style of this project.
  • [] My change requires a change to the documentation.
  • [] I have updated the documentation accordingly. (N/A)
  • [] I have updated the tests accordingly. (N/A)

Copy link
Owner

@matchai matchai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch @dubnev!
Thank you for your contribution. 😄

@matchai matchai merged commit 7c2a1d5 into matchai:master Feb 17, 2019
matchai pushed a commit that referenced this pull request Feb 17, 2019
## [2.2.3](v2.2.2...v2.2.3) (2019-02-17)

### Bug Fixes

* git status section shows the correct symbol for unmerged ([#159](#159)) ([7c2a1d5](7c2a1d5))
@matchai
Copy link
Owner

matchai commented Feb 17, 2019

🎉 This PR is included in version 2.2.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@matchai
Copy link
Owner

matchai commented Mar 2, 2019

@all-contributors please add @dubnev for bug

@allcontributors
Copy link
Contributor

@matchai

I've put up a pull request to add @dubnev! 🎉

matchai pushed a commit that referenced this pull request Mar 2, 2019
Adds @dubnev as a contributor for bug.

This was requested by matchai [in this comment](#159 (comment))

Co-authored-by: null <allcontributors[bot]@users.noreply.github.com>
adamsimp pushed a commit to adamsimp/spacefish that referenced this pull request May 6, 2019
Adds @dubnev as a contributor for bug.

This was requested by matchai [in this comment](matchai#159 (comment))

Co-authored-by: null <allcontributors[bot]@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants