#31504 closed enhancement (fixed)

details in relint config

Reported by: Frédéric Chapoton Owned by:
Priority: major Milestone: sage-9.3
Component: scripts Keywords:
Cc: Tobias Diez Merged in:
Authors: Frédéric Chapoton Reviewers: Matthias Koeppe
Report Upstream: N/A Work issues:
Branch: ac3838b (Commits, GitHub, GitLab) Commit: ac3838b17e6afe0d075a97bd3786ae9c65be7a31
Dependencies: Stopgaps:

Status badges

Description

  • do no test for "Returns", as there are too many of them in the codebase
  • remove the check for python2-only print syntax

Change History (3)

comment:1 Changed 21 months ago by Frédéric Chapoton

Branch: u/chapoton/31504
Commit: ac3838b17e6afe0d075a97bd3786ae9c65be7a31
Status: newneeds_review

New commits:

ac3838badjust the relint configuration file

comment:2 Changed 21 months ago by Matthias Köppe

Reviewers: Matthias Koeppe
Status: needs_reviewpositive_review

Fine with me.

Would you have a moment to add a summary of your coding style work to https://wiki.sagemath.org/ReleaseTours/sage-9.3#Cleaning_of_the_Sage_codebase_to_conform_to_best_practices?

comment:3 Changed 21 months ago by Volker Braun

Branch: u/chapoton/31504ac3838b17e6afe0d075a97bd3786ae9c65be7a31
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.