Skip to content

Conversation

@AlgoFoe
Copy link
Contributor

@AlgoFoe AlgoFoe commented Jan 3, 2026

Description

What is this PR

  • Bug fix
  • Addition of a new feature
  • Other (configuration)

Why is this PR needed?

  • When running tests via tox, CI-related environment variables such as CI and GITHUB_ACTIONS are not available inside the tox environment by default.

What does this PR do?

  • It adds CI and GITHUB_ACTIONS to the passenv configuration in the legacy_tox_ini section.
    As a result, these standard CI environment variables are forwarded into tox test environments.

References

How has this PR been tested?

  • Tests were run locally using pytest

Is this a breaking change?

  • No.

Does this PR require an update to the documentation?

  • No.

Checklist:

  • The code has been tested locally
  • Tests have been added to cover all new functionality
  • The documentation has been updated to reflect any changes
  • The code has been formatted with pre-commit

Copy link
Member

@IgorTatarnikov IgorTatarnikov left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

Thank you!

@IgorTatarnikov IgorTatarnikov merged commit f5117ae into neuroinformatics-unit:main Jan 5, 2026
10 checks passed
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