Opened 9 months ago

Closed 9 months ago

#26568 closed enhancement (fixed)

clean c_graph.pyx

Reported by: dcoudert Owned by:
Priority: major Milestone: sage-8.5
Component: graph theory Keywords: py3, graph
Cc: tscrim Merged in:
Authors: David Coudert Reviewers: Travis Scrimshaw
Report Upstream: N/A Work issues:
Branch: 859fea9 (Commits) Commit: 859fea9a80a44645dfdf25862c7a3a8021023fea
Dependencies: Stopgaps:

Description (last modified by dcoudert)

PEP8 cleaning of the file.

Change History (8)

comment:1 Changed 9 months ago by dcoudert

  • Branch set to public/26568_c_graph_pyx
  • Cc tscrim added
  • Commit set to bffda26f038b4d9e2ebbbc2aba31b92a8a15ee1c
  • Description modified (diff)
  • Status changed from new to needs_review

New commits:

bffda26trac #26568: clean c_graph.pyx

comment:2 Changed 9 months ago by tscrim

  • Reviewers set to Travis Scrimshaw
  • Status changed from needs_review to positive_review

LGTM.

comment:3 Changed 9 months ago by vbraun

  • Status changed from positive_review to needs_work

Merge conflict

comment:4 Changed 9 months ago by dcoudert

I guess I have to wait until next beta as I don't see any merge conflict.

comment:5 Changed 9 months ago by git

  • Commit changed from bffda26f038b4d9e2ebbbc2aba31b92a8a15ee1c to 859fea9a80a44645dfdf25862c7a3a8021023fea

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

859fea9trac #26568: fix merge conflict with 8.5.beta2

comment:6 Changed 9 months ago by dcoudert

  • Status changed from needs_work to needs_review

I fixed the merge conflict with 8.5.beta2

comment:7 Changed 9 months ago by tscrim

  • Status changed from needs_review to positive_review

comment:8 Changed 9 months ago by vbraun

  • Branch changed from public/26568_c_graph_pyx to 859fea9a80a44645dfdf25862c7a3a8021023fea
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.