Skip to content

add dupline test, currently failing - #2

Closed
itdependsnetworks wants to merge 1 commit into
developfrom
dupline
Closed

add dupline test, currently failing#2
itdependsnetworks wants to merge 1 commit into
developfrom
dupline

Conversation

@itdependsnetworks

Copy link
Copy Markdown
Contributor

This test fails, and demonstrates the issue identified in #1

@jeffkala

Copy link
Copy Markdown
Collaborator

@itdependsnetworks I'm confused on the verbiage. What does currently failing indicate does this PR fix it or is this still a WIP.

@itdependsnetworks

Copy link
Copy Markdown
ContributorAuthor

sorry, should have tagged @jmcgill298 to take a look. He may have already addressed this. This is a test to demonstrate condition that is currently not acting appropriately. So, no action right now @jeffkala

@jeffkala

Copy link
Copy Markdown
Collaborator

duplicate line was discussed and there hasn't been a valid configuration use case so far to support duplicate lines at the root level of a configuration. It should be documented.

@itdependsnetworks

Copy link
Copy Markdown
ContributorAuthor

Addressed in #128

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@itdependsnetworks@jeffkala