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

Unpin jinja2 requirements #75

Merged
merged 6 commits into from
Feb 1, 2024
Merged

Unpin jinja2 requirements #75

merged 6 commits into from
Feb 1, 2024

Conversation

NilashishC
Copy link
Collaborator

No description provided.

@NilashishC NilashishC requested a review from a team as a code owner January 29, 2024 13:23
@NilashishC NilashishC requested review from ssbarnea and shatakshiiii and removed request for a team January 29, 2024 13:23
@NilashishC NilashishC added the skip-changelog Skip changelog check when this label is found. label Jan 29, 2024
@ssbarnea ssbarnea changed the title Bump and unpin jinja2 requirement to >=3.1.3 Unpin jinja2 requirements Jan 31, 2024
@ssbarnea ssbarnea added bug Something isn't working and removed skip-changelog Skip changelog check when this label is found. labels Jan 31, 2024
.pre-commit-config.yaml Outdated Show resolved Hide resolved
@@ -48,6 +50,7 @@ commands =

[testenv:lint]
description = Enforce quality standards under {basepython} ({envpython})
base_python = python3.9
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
base_python = python3.9
base_python = python3.10

since 3.9 support has been removed

optional-dependencies.docs = {file = [".config/docs-requirements.txt"]}
optional-dependencies.test = {file = [".config/requirements-test.txt"]}
dependencies = {file = ".config/requirements.in"}
optional-dependencies.docs = {file = [".config/docs-requirements.in"]}
Copy link
Contributor

Choose a reason for hiding this comment

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

@NilashishC just to keep the same pattern as other projects, could you please rename to requirements-docs.in and requirements-test.in

entry: pip-compile -q --no-annotate --output-file=.config/constraints.txt pyproject.toml --all-extras --strip-extras
files: ^.config\/.*(requirements|constraints).*$
language: python
language_version: "3.9" # minimal we support officially
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
language_version: "3.9" # minimal we support officially
language_version: "3.10" # minimal we support officially

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants