Skip to content

Conversation

@KristopherKubicki
Copy link
Member

Summary

  • make pytest run via Python module so tests import local modules
  • update README badges to use the .github repo
  • update profile README badges as well
  • badges now reference the GapIntelligence organization

Testing

  • python -m pytest -q

https://chatgpt.com/codex/tasks/task_e_6858160391d88333bcf7bf134d7fda32

Copilot AI review requested due to automatic review settings June 22, 2025 14:51
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR ensures CI tests import local modules by running pytest as a module and updates repository badges to reference the GapIntelligence organization’s .github repo.

  • Switch test command to python -m pytest for proper module imports
  • Update CI and codecov badges in both root and profile READMEs to point at GapIntelligence/.github

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
profile/README.md Updated CI and codecov badge URLs to use GapIntelligence/.github
README.md Updated CI and codecov badge URLs to use GapIntelligence/.github
.github/workflows/ci.yml Changed test step to run python -m pytest instead of pytest

@codecov
Copy link

codecov bot commented Jun 22, 2025

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

ℹ️ You can also turn on project coverage checks and project coverage reporting on Pull Request comment

Thanks for integrating Codecov - We've got you covered ☂️

@KristopherKubicki KristopherKubicki merged commit 818c0ca into staging Jun 22, 2025
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants