D1021: check-code: forbid platform.system()
This revision was automatically updated to reflect the committed changes. Closed by commit rHGa679aa582d8d: check-code: forbid platform.system() (authored by quark, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D1021?vs=2634=2680 REVISION DETAIL https://phab.mercurial-scm.org/D1021 AFFECTED FILES contrib/check-code.py CHANGE DETAILS diff --git a/contrib/check-code.py b/contrib/check-code.py --- a/contrib/check-code.py +++ b/contrib/check-code.py @@ -362,6 +362,7 @@ (r'\.next\(\)', "don't use .next(), use next(...)"), (r'([a-z]*).revision\(\1\.node\(', "don't convert rev to node before passing to revision(nodeorrev)"), +(r'platform\.system\(\)', "don't use platform.system(), use pycompat"), # rules depending on implementation of repquote() (r' x+[xpqo%APM][\'"]\n\s+[\'"]x', To: quark, #hg-reviewers, lothiraldan Cc: ryanmce, lothiraldan, mercurial-devel ___ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel
D1021: check-code: forbid platform.system()
quark updated this revision to Diff 2634. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D1021?vs=2608=2634 REVISION DETAIL https://phab.mercurial-scm.org/D1021 AFFECTED FILES contrib/check-code.py CHANGE DETAILS diff --git a/contrib/check-code.py b/contrib/check-code.py --- a/contrib/check-code.py +++ b/contrib/check-code.py @@ -362,6 +362,7 @@ (r'\.next\(\)', "don't use .next(), use next(...)"), (r'([a-z]*).revision\(\1\.node\(', "don't convert rev to node before passing to revision(nodeorrev)"), +(r'platform\.system\(\)', "don't use platform.system(), use pycompat"), # rules depending on implementation of repquote() (r' x+[xpqo%APM][\'"]\n\s+[\'"]x', To: quark, #hg-reviewers, lothiraldan Cc: ryanmce, lothiraldan, mercurial-devel ___ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel
D1021: check-code: forbid platform.system()
ryanmce added a comment. +1 to this change, happy to land this once my comments on the series are addressed REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D1021 To: quark, #hg-reviewers, lothiraldan Cc: ryanmce, lothiraldan, mercurial-devel ___ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel
D1021: check-code: forbid platform.system()
lothiraldan accepted this revision. lothiraldan added a comment. LGTM but I can't validate the Mac OS and Windows tests. REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D1021 To: quark, #hg-reviewers, lothiraldan Cc: lothiraldan, mercurial-devel ___ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel
D1021: check-code: forbid platform.system()
quark created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY See the previous patches for the reason. REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D1021 AFFECTED FILES contrib/check-code.py CHANGE DETAILS diff --git a/contrib/check-code.py b/contrib/check-code.py --- a/contrib/check-code.py +++ b/contrib/check-code.py @@ -362,6 +362,7 @@ (r'\.next\(\)', "don't use .next(), use next(...)"), (r'([a-z]*).revision\(\1\.node\(', "don't convert rev to node before passing to revision(nodeorrev)"), +(r'platform\.system()', "don't use platform.system(), use pycompat"), # rules depending on implementation of repquote() (r' x+[xpqo%APM][\'"]\n\s+[\'"]x', To: quark, #hg-reviewers Cc: mercurial-devel ___ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel