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.

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

Fix reporters not working #4

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

Conversation

rdmueller
Copy link
Member

@rdmueller rdmueller commented Jan 14, 2025

Fixes #3

Update ConsoleReporter class to correctly handle findings and display their positions.

  • Change ConsoleReporter class in asciidoc_linter/reporter.py to use error.position.line instead of error.line to correctly access the line number.
  • Ensure ConsoleReporter class correctly handles the position attribute of Finding instances.

For more details, open the Copilot Workspace session.

Fixes #3

Update `ConsoleReporter` class to correctly handle findings and display their positions.

* Change `ConsoleReporter` class in `asciidoc_linter/reporter.py` to use `error.position.line` instead of `error.line` to correctly access the line number.
* Ensure `ConsoleReporter` class correctly handles the `position` attribute of `Finding` instances.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/docToolchain/asciidoc-linter/issues/3?shareId=XXXX-XXXX-XXXX-XXXX).
@A-n-d-y-K A-n-d-y-K mentioned this pull request Jan 24, 2025
Sign up for free to 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.

Reporters not working
1 participant