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

Update jinja2: 2.11.3 -> 3.0.3 #303

Open
wants to merge 1 commit into
base: stable
Choose a base branch
from

Conversation

sei40kr
Copy link

@sei40kr sei40kr commented Nov 18, 2024

Why is this change needed?

NixOSから使いたいのでNixpkgsにatcoder-toolsを登録したい。
しかし、Nixpkgsには既にJinja2 3.0.0以降のバージョンしか存在しないため、post-checkでエラーになる。
そのため、atcoder-toolsのjinja2を ^3.0.0 に移行したい。

What did you implement?

  • Jinja2を2.11.3 → 3.0.3 にアップデート
  • autoescape を明示的に False

autoescape に関してはLLMに質問したところ、Jinja2 3.0.0から autoescape=True がデフォルトになるとのことで明示的に変更したのですが、Jinja2のchangelogを読んでも該当箇所を見つけられず。
もし、autoescape=False で試して問題なければ冗長なのでこの変更は取り消します。

autoescape=False で試しても <,> の部分はエスケープされませんでした。
よって明示的に autoescape=False を指定する必要はないため削除しました。

What behavior do you expect?

  • 既存の挙動が変わらないこと

手元では

atcoder-tools --lang rust --without-login dp

して

  • 正常に終了すること
  • コードテンプレートが作成されていること
  • Rustの型引数の部分の <,> がエスケープされていないこと

を確認しました。

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