Page MenuHomeSolus
Differential D2884

Updated pylint to 1.8.4
ClosedPublic
Actions

Authored by debug on May 7 2018, 8:45 PM.
Tags
None
Referenced Files
F11051960: D2884.id.diff
Thu, Aug 10, 8:31 PM
F11051959: D2884.id7081.diff
Thu, Aug 10, 8:31 PM
F11051958: D2884.id7082.diff
Thu, Aug 10, 8:31 PM
F11051957: D2884.id7593.diff
Thu, Aug 10, 8:31 PM
F11051955: D2884.id7078.diff
Thu, Aug 10, 8:31 PM
F11051954: D2884.id7087.diff
Thu, Aug 10, 8:31 PM
F11036422: D2884.diff
Wed, Aug 9, 7:19 PM
F11018190: D2884.id7078.diff
Sun, Aug 6, 9:58 AM
View All 27 Files
Subscribers
DataDrake
kyrios123

Details

Reviewers
DataDrake
Group Reviewers
Triage Team
Commits
R3714:7d66af8d9232: Updated pylint to 1.8.4
Summary

Updated pylint to the latest version 1.8.4.
Change log available here.

Test Plan

Installed in a virtual machine with the updated version of python-astroid and ran with Python files with/without errors.
All tests ran as expected.

Diff Detail

Repository
R3714 pylint
Branch
master
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

debug created this revision.May 7 2018, 8:45 PM
Herald added a reviewer: Triage Team. · View Herald TranscriptMay 7 2018, 8:45 PM
debug requested review of this revision.May 7 2018, 8:45 PM
debug added a comment.May 8 2018, 1:05 AM

After looking back at this differential I believe I misunderstood when to use 'Depends'. Updating 'pylint' does not *require* updating 'python-astroid' however as it states of the GitHub page the later the version of 'python-astroid' the better. I'll leave the 'Depends' in the differential for now unless instructed otherwise.

kyrios123 added a subscriber: kyrios123.May 8 2018, 9:37 AM
kyrios123 added inline comments.
package.yml
6

General comment, when you update a package, please make sure to adapt the license code to SPDX 3.x.
The impacted licenses are mainly the AGPL, GPL, GDFL, LGPL that must be suffixed with -only or -or-later.

debug added a comment.May 8 2018, 9:43 AM

Okay. I will go through my current open patches and update them where necessary.

debug updated this revision to Diff 7081.May 8 2018, 10:21 AM

Updated license

debug updated this revision to Diff 7082.May 8 2018, 10:26 AM

Forgot to update license in pspec_x86_64.xml

debug updated this revision to Diff 7087.May 8 2018, 3:14 PM
debug marked an inline comment as done.
debug edited the summary of this revision. (Show Details)

Removed the depends on D2885.

DataDrake accepted this revision.Jun 10 2018, 4:46 PM
DataDrake added a subscriber: DataDrake.

LGTM. Thanks!

This revision is now accepted and ready to land.Jun 10 2018, 4:46 PM
Closed by commit R3714:7d66af8d9232: Updated pylint to 1.8.4 (authored by JamesLee, committed by DataDrake). · Explain WhyJun 10 2018, 4:53 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
8 lines
38 lines

Diff 7078

View Options

package.yml

Loading...
View Options

pspec_x86_64.xml

Loading...
Copyright © 2015-2021 Solus Project. The Solus logo is Copyright © 2016-2021 Solus Project. All Rights Reserved.