Opened 3 years ago

Closed 2 years ago

#27138 closed enhancement (fixed)

pep8 in digraph_generators.py (part 5)

Reported by: dcoudert Owned by:
Priority: minor Milestone: sage-8.8
Component: graph theory Keywords:
Cc: Merged in:
Authors: David Coudert Reviewers: Frédéric Chapoton
Report Upstream: N/A Work issues:
Branch: 4c4c98b (Commits, GitHub, GitLab) Commit: 4c4c98b47ee121241b697b913ad95c761652f7cb
Dependencies: Stopgaps:

Status badges

Description

Clean documentation of DiGraphGenerators and method __call__.

Change History (8)

comment:1 Changed 3 years ago by dcoudert

  • Branch set to u/dcoudert/27138_digraphs_generators_5
  • Commit set to 997c790ff2fccd6e9929acfd239e254d2c21acc8
  • Status changed from new to needs_review

New commits:

​997c790trac #27138: clean digraphs_generators part 5

comment:2 Changed 3 years ago by dcoudert

  • Priority changed from major to minor

Together with #27133, #27134, #27135, #27136 and #27137, file digraph_generators.py is cleaned.

comment:3 Changed 2 years ago by embray

  • Milestone changed from sage-8.7 to sage-8.8

Ticket retargeted after milestone closed (if you don't believe this ticket is appropriate for the Sage 8.8 release please retarget manually)

comment:4 Changed 2 years ago by git

  • Commit changed from 997c790ff2fccd6e9929acfd239e254d2c21acc8 to 4c4c98b47ee121241b697b913ad95c761652f7cb

Branch pushed to git repo; I updated commit sha1. New commits:

​4c4c98btrac #27138: fix merge conflict in index.rst

comment:5 Changed 2 years ago by dcoudert

rebase on 8.8.beta3 and fix merge conflict.

comment:6 Changed 2 years ago by chapoton

  • Reviewers set to Frédéric Chapoton
  • Status changed from needs_review to positive_review

ok

comment:7 Changed 2 years ago by dcoudert

Thank you !

comment:8 Changed 2 years ago by vbraun

  • Branch changed from u/dcoudert/27138_digraphs_generators_5 to 4c4c98b47ee121241b697b913ad95c761652f7cb
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.