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.

Sign up for GitHub

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

Jump to bottom

handle unnamed lists in merge_lists function #25

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

handle unnamed lists in merge_lists function #25

wants to merge 1 commit into from

Conversation

vpanfilov
Copy link

Unnamed lists can now be overriden by another config section (fix #24)

@vpanfilov
handle unnamed lists in merge_lists function
33bfb0b 
Unnamed lists can now be overriden by another config section (fix rstudio#24)
jjallaire
jjallaire requested changes Oct 11, 2019
View reviewed changes
Copy link
Member

@jjallaire jjallaire left a comment

Choose a reason for hiding this comment

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

Could you add a test for this new behavior? Thanks!

@vpanfilov
Copy link
Author

Ok, I will provide some basic tests when I have some free time. At first, I want to take a look at others YAML merge utilities and compare their behaviour.

@jankowtf jankowtf mentioned this pull request Jan 4, 2021
Open
@andrie andrie changed the base branch from master to main May 14, 2021 07:16
@psychelzh psychelzh mentioned this pull request Apr 26, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jjallaire jjallaire jjallaire requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Merging unnamed lists gives wrong result
2 participants
@vpanfilov @jjallaire