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

style: adjust styles to pass hatch run check #1225

Open
wants to merge 4 commits into
base: feature/tooling
Choose a base branch
from

Conversation

KarolJagodzinski
Copy link
Contributor

@KarolJagodzinski KarolJagodzinski commented Feb 3, 2025

result of last command from
hatch run check

image

@KarolJagodzinski KarolJagodzinski force-pushed the feature/tooling_resolve_conflicts branch from bd80393 to 2e2bb01 Compare February 3, 2025 18:43
@KarolJagodzinski KarolJagodzinski force-pushed the feature/tooling_resolve_conflicts branch 5 times, most recently from 4fa3f3b to 64e6cdc Compare February 4, 2025 08:57
@KarolJagodzinski KarolJagodzinski force-pushed the feature/tooling_resolve_conflicts branch 5 times, most recently from c665476 to 0aacb72 Compare February 14, 2025 08:38
@KarolJagodzinski
Copy link
Contributor Author

There is an alternative to run integrations tests with following configuration:

## Define a matrix environment for integration tests
[tool.hatch.envs.test-integration]
features = ["tests"]

[[tool.hatch.envs.test-integration.matrix]]
framework = ["fastapi", "flask", "starlette"]

[tool.hatch.envs.test-integration.scripts]
test = "pip install {matrix:framework} && pytest tests_integrations/{matrix:framework}"

but I like the idea of having it separated on github workflow to make it more visible of what is happening.

any thoughts?

@KarolJagodzinski KarolJagodzinski force-pushed the feature/tooling_resolve_conflicts branch 2 times, most recently from 9934944 to fc4ab9a Compare February 14, 2025 10:25
@KarolJagodzinski KarolJagodzinski force-pushed the feature/tooling_resolve_conflicts branch from fc4ab9a to 4970de5 Compare February 14, 2025 10:50
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.

1 participant