- User Since
- Sep 1 2021, 8:32 PM (6 w, 6 d)
Tue, Oct 5
Works for me! 🙂 I didn't realize you can pull the info automatically from arc. Thanks!
Mon, Oct 4
Thanks! Could you help me commit again when you have time? (same info as here)
Mon, Sep 27
Sep 14 2021
Thanks for your reply and sorry about my very sluggish reply!
I am looking into updating the docs as you suggested, and that got me looking at this doc page. Interestingly, that doc page's version of the add_new_check.py template doesn't have the Note diagnostic. So that got me digging into the history.
It seems like 2 commits are at odds with each other about what the right thing to do is with FixItHints w.r.t. Note vs Warning diagnostics.
Sep 10 2021
Thanks! I don't have commit access, I was just reading here what to do:
Prior to obtaining commit access, it is common practice to request that someone with commit access commits on your behalf. When doing so, please provide the name and email address you would like to use in the Author property of the commit.
Sep 3 2021
Ah, and to your direct question:
Sep 2 2021
No worries! Figured I'd start the conversation with the easy fix, but was fully prepared to hear that we can't let go of python 2 just yet, as much as we may want to :).
Let me know if this fix looks suitable. Thanks for your time!
Reverted python-3-only enforcement, fixed both scripts to work with both python 2 and python 3
(generally based on the guidance here: https://python-future.org/compatible_idioms.html#strings-and-bytes)