+
Skip to content

Conversation

nelsonni
Copy link
Member

The results of merging can have varied outcomes that need to be properly handled, including:

To properly model these scenarios, this PR introduces processMergeOutput function for evaluating the raw output from an executed merge in order to discern status: Passing and status: Failing as well as reasons for these results.

This PR resolves #978.

Changes:

This PR makes the following changes:

  • Rename TimelineButtons component to MergeButtons to reflect usage being dependent on merging
  • Add git-rev-list command for listing commit objects in reverse chronological order
  • Add processMergeOutput function for populating status, alreadyMerged, fastForward, output, mergeCommit, mergeStrategy, and conflicts fields from a merge
  • Add test specs for processMergeOutput
  • Add MergeStatus component for displaying success, warning, and error alert messages based on merge results
  • Add React Developer Tools hook to Electron extensions

@nelsonni nelsonni added bug Bug reports or bug fixes feature Feature requests or improvements labels Dec 16, 2022
@nelsonni nelsonni self-assigned this Dec 16, 2022
@nelsonni nelsonni merged commit 70de58a into main Dec 16, 2022
@nelsonni nelsonni deleted the fix/merge-status branch December 16, 2022 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Bug reports or bug fixes feature Feature requests or improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

error: Your local changes to the following files would be overwritten by merge

1 participant

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载