Skip to content

Conversation

@JonZeolla
Copy link
Member

@JonZeolla JonZeolla commented Aug 20, 2025

Contributor Comments

This adds the slow marker to our pytest configs in pyproject.toml; required to cleanup a warning as seen here:

=============================== warnings summary ===============================
tests/test_cookiecutter.py:215
  /home/runner/work/ai-native-python/ai-native-python/tests/test_cookiecutter.py:215: PytestUnknownMarkWarning: Unknown pytest.mark.slow - is this a typo?  You can register custom marks to avoid this warning - for details, see https://docs.pytest.org/en/stable/how-to/mark.html
    @pytest.mark.slow

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
=========================== short test summary info ============================
FAILED tests/test_cookiecutter.py::test_default_project - Failed: Command failed with exit code 1. See output above.
====== 1 failed, 2 passed, 36 deselected, 1 warning in 469.09s (0:07:49) =======

This also runs a task update to fix the tests which were previously failing (as shown above)

It also updates the code of conduct to work around a link validation issue with lychee which was changed upstream

Pull Request Checklist

Thank you for submitting a contribution!

Please address the following items:

  • If you are adding a dependency, please explain how it was chosen.
  • If manual testing is needed in order to validate the changes, provide a testing plan and the expected results.
  • Validate that documentation is accurate and aligned to any project updates or additions.

@JonZeolla JonZeolla enabled auto-merge (squash) August 20, 2025 16:16
Copy link

@ai-coding-guardrails ai-coding-guardrails bot left a comment

Choose a reason for hiding this comment

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

Nice work! 😎

I didn't find anything of concern

Reviewed with 🤟 by Zenable

@JonZeolla JonZeolla changed the title chore(testing): configure the slow marker chore(testing): cleanup test running Aug 20, 2025
@JonZeolla JonZeolla merged commit 1fda4f2 into main Aug 20, 2025
7 checks passed
@JonZeolla JonZeolla deleted the fix/config/pytest-markers branch August 20, 2025 17:15
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.

2 participants