cli icon indicating copy to clipboard operation
cli copied to clipboard

refactor: use with statement for file reading

Open allrob23 opened this issue 8 months ago • 3 comments

Changes

This PR updates the file reading logic to use a with statement, aligning with Python best practices for resource management. is a minor change but ensures consistency with the project standard, as other parts already use with statements.

allrob23 avatar Apr 21 '25 18:04 allrob23

Hi @allrob23 ! Thanks for your contribution. First-time contributors are required to sign CLA in order for their change to be accepted. If this is something you're willing to do, please reach out to [email protected] and we'll take it from there.

Thank you!

andrewnester avatar Apr 22 '25 08:04 andrewnester

Hi @andrewnester, I just sent the email.

allrob23 avatar Apr 22 '25 10:04 allrob23

Hi @andrewnester, just dropping by to let you know that I signed the CLA

allrob23 avatar May 05 '25 18:05 allrob23

An authorized user can trigger integration tests manually by following the instructions below:

Trigger: go/deco-tests-run/cli

Inputs:

  • PR number: 2740
  • Commit SHA: 3f1c9efcc8e2c3132c36d1ab781d3e419d4d4c87

Checks will be approved automatically on success.

github-actions[bot] avatar May 07 '25 10:05 github-actions[bot]