Opened 4 years ago

Closed 4 years ago

#17923 closed enhancement (fixed)

Better document python's sys_path_security patch

Reported by: Snark Owned by:
Priority: trivial Milestone: sage-6.6
Component: documentation Keywords:
Cc: Merged in:
Authors: Julien Puydt Reviewers: Volker Braun
Report Upstream: N/A Work issues:
Branch: ece5a8b (Commits) Commit: ece5a8bef34379685c01bb75dbebb4a494a6932b
Dependencies: Stopgaps:

Description

This patch needs more explaining.

Change History (7)

comment:1 Changed 4 years ago by Snark

  • Branch set to u/Snark/better_document_python_sys_path_patch

comment:2 Changed 4 years ago by Snark

  • Commit set to ece5a8bef34379685c01bb75dbebb4a494a6932b
  • Status changed from new to needs_review

New commits:

ece5a8bBetter document the sys_path_security patch of the python package

comment:3 Changed 4 years ago by vbraun

  • Reviewers set to Volker Braun

Looks good to me, if you can fill in your Author: name then you can set it to positive review.

comment:4 Changed 4 years ago by Snark

  • Authors set to Julien Puydt
  • Reviewers Volker Braun deleted

comment:5 Changed 4 years ago by Snark

  • Reviewers set to Volker Braun

Hmmm... I don't understand why it reverted Volker as a reviewer... putting him back.

comment:6 Changed 4 years ago by mmezzarobba

  • Status changed from needs_review to positive_review

See Volker's comment.

comment:7 Changed 4 years ago by vbraun

  • Branch changed from u/Snark/better_document_python_sys_path_patch to ece5a8bef34379685c01bb75dbebb4a494a6932b
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.