Skip to content

Conversation

@sofietoft
Copy link
Contributor

Description

Added a new article about how to resolve merge conflicts with flexible environments.

Also fixed some environment refs in other articles while waiting for deployments to complete.

@sofietoft sofietoft marked this pull request as draft April 7, 2025 13:19
@sofietoft sofietoft marked this pull request as ready for review April 7, 2025 13:30

* The log file containing a list of the conflicting files.
* The clone link for the flexible environment.
* The clone link for the mainline environment that flexible environment is connected to.
Copy link
Member

Choose a reason for hiding this comment

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

Conflicts can happen between any environment, that is merging together.

Would it make sense to call them source and destination environment?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This guide is specifically for merge conflicts between a flex env and a mainline env.
We already have a guide on how to resolve merge conflicts between mainline environments.

If the process is the same, then we need to merge the two articles and rewrite it.
This might not be possible to get done before Friday.

Copy link
Contributor

@eshanrnh eshanrnh left a comment

Choose a reason for hiding this comment

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

Looks good to me, @sofietoft 🙌 Just a couple of typo suggestions here and there,,,

sofietoft and others added 10 commits April 9, 2025 10:09
Co-authored-by: Esha Noronha <82437098+eshanrnh@users.noreply.github.com>
Co-authored-by: Esha Noronha <82437098+eshanrnh@users.noreply.github.com>
Co-authored-by: Esha Noronha <82437098+eshanrnh@users.noreply.github.com>
Co-authored-by: Esha Noronha <82437098+eshanrnh@users.noreply.github.com>
Co-authored-by: Esha Noronha <82437098+eshanrnh@users.noreply.github.com>
Co-authored-by: Esha Noronha <82437098+eshanrnh@users.noreply.github.com>
Co-authored-by: Esha Noronha <82437098+eshanrnh@users.noreply.github.com>
Co-authored-by: Esha Noronha <82437098+eshanrnh@users.noreply.github.com>
Co-authored-by: Esha Noronha <82437098+eshanrnh@users.noreply.github.com>
@sofietoft sofietoft requested a review from eshanrnh April 9, 2025 17:09
@sofietoft
Copy link
Contributor Author

sofietoft commented Apr 9, 2025

@eshanrnh - I've updated the list under "Preparations".
I've added source and target in parenthesis - can you give it a look, and tell me if it makes sense to do? 🤔

@eshanrnh
Copy link
Contributor

I've added my thoughts, @sofietoft 🙌

Co-authored-by: Esha Noronha <82437098+eshanrnh@users.noreply.github.com>
@sofietoft sofietoft merged commit a5027d9 into cloud/flexibleEnvironment Apr 10, 2025
2 checks passed
@sofietoft sofietoft deleted the flex/mergeConflicts branch April 28, 2025 07:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants