operator icon indicating copy to clipboard operation
operator copied to clipboard

Change testing/* to use single quotes

Open tonyandrewmeyer opened this issue 1 year ago • 1 comments

To be consistent with the rest of the code, we should change all of testing/**/*.py to use ' in preference to ", as we did with the other code earlier in the year.

The testing/pyproject.toml configuration should be altered to remove the special case, then ruff can be run, and then we'll want to add the commit to the blame ignore list.

tonyandrewmeyer avatar Oct 10 '24 02:10 tonyandrewmeyer

bike shed time: let's take lead from scneario and switch to double quotes 🙄

dimaqq avatar Oct 11 '24 03:10 dimaqq

@james-garner-canonical is hoping to look at this in the next couple of weeks.

benhoyt avatar Mar 25 '25 01:03 benhoyt

@james-garner-canonical this was completed in https://github.com/canonical/operator/pull/1691, no?

tonyandrewmeyer avatar Jun 03 '25 23:06 tonyandrewmeyer

Yes, thanks for catching that this was still open.

james-garner-canonical avatar Jun 03 '25 23:06 james-garner-canonical