-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Fix: Git and GitHub Collabo | 2nd Issue Ticket Task | Working with GitHub web UI #8
Merged
Merged
Changes from 2 commits
Commits
Show all changes
4 commits
Select commit
Hold shift + click to select a range
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here's the next step that concerns conflict resolution @BA-atomic
How to know that a conflict has occurred - Look towards the bottom of this pull request, you will this text on the UI: "This branch has conflicts that must be resolved". Also notice that the file you last made changes to was mentioned - That is where the conflict took place.
Screenshot
What caused the conflict? - Someone (me 😉), who still had the version of the file that contains the old content (Inline Code Block:
This code block text will show up inline
.), made changes to that same line so that it now says: Inline Code Block:This code block text will show up inline
. Do you notice the difference?.Guess what? Maintainers of this repo already added/merged the above change into the original Collabo Community repo's develop branch!
You on the other hand, have also made changes to the same line of the same file. The content that both of you added at the end of that line are totally different. You also added new content somewhere within the line, that the other person did not have before it was merged.
In summary, both of you who made changes to that same line are not in agreement i.e. there's a "conflict".
What to do next?
Here is the correct content that should be on the "Inline Code Block (with non-code text)" line:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@BA-atomic what you added is not correct check the What to do next? section of the conflict resolution instructions given, to see was you missed.