Skip to content

Linter allows overlapping statements #26543

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.

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

Open
1 of 2 tasks
caugner opened this issue Apr 18, 2025 · 0 comments · May be fixed by #26544
Open
1 of 2 tasks

Linter allows overlapping statements #26543

caugner opened this issue Apr 18, 2025 · 0 comments · May be fixed by #26544
Assignees
Labels
linter Issues or pull requests regarding the tests / linter of the JSON files. p1 High priority – Should be addressed in the next few releases.

Comments

@caugner
Copy link
Contributor

caugner commented Apr 18, 2025

What type of issue is this?

Infrastructure issue

What is the issue?

The following Edge statement does not cause a Linter issue:

"edge": [
  {
    "version_added": "79"
  },
  {
    "version_added": "12",
    "partial_implementation": true
  }
]

What behavior were you expecting?

The Linter should fail, and suggest adding version_removed to the second statement.

Maybe this could be auto-fixable.

What version(s) of BCD is the issue present in?

  • The current BCD release
  • The current version of the main branch

Do you have anything more you want to share?

Noticed in #26542 (review).

@caugner caugner added the needs triage This issue needs to be confirmed label Apr 18, 2025
@caugner caugner linked a pull request Apr 18, 2025 that will close this issue
@caugner caugner changed the title Linter allows active partial and non-partial statements Linter allows overlapping statements Apr 18, 2025
@queengooborg queengooborg added the linter Issues or pull requests regarding the tests / linter of the JSON files. label Apr 21, 2025
@caugner caugner added p1 High priority – Should be addressed in the next few releases. p2 Medium priority – Community PRs highly encouraged. and removed needs triage This issue needs to be confirmed p2 Medium priority – Community PRs highly encouraged. labels Apr 22, 2025
@caugner caugner self-assigned this Apr 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
linter Issues or pull requests regarding the tests / linter of the JSON files. p1 High priority – Should be addressed in the next few releases.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants