Closed Bug 1633053 Opened 5 years ago Closed 2 years ago

Fix/enable ESLint react/prop-types

Categories

(Tree Management :: Treeherder: Frontend, enhancement, P3)

enhancement

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: camd, Assigned: bolah2009)

Details

Attachments

(1 file)

We should fix the violations and enable this rule.

I would like to work on it if that's okay.

Flags: needinfo?(cdawson)

Bola-- So sorry I had such a slow response to you on this! Yes, please work on this. That would be very much appreciated! :)

Assignee: nobody → bola_ahmed2002
Flags: needinfo?(cdawson)
Priority: -- → P3

Thanks, Cameron, I already started working on it pending your response.
However, the errors are many (500+) and span various files. Is it okay to open a single PR for this?

Flags: needinfo?(cdawson)

The list of files (31) that have been modified so far with 300+ errors left:

  • ui/intermittent-failures/GraphAlternateView.jsx
  • ui/job-view/CustomJobActions.jsx
  • ui/job-view/KeyboardShortcuts.jsx
  • ui/job-view/details/BugFiler.jsx
  • ui/job-view/details/DetailsPanel.jsx
  • ui/job-view/details/PinBoard.jsx
  • ui/job-view/details/summary/ActionBar.jsx
  • ui/job-view/details/summary/ClassificationsPanel.jsx
  • ui/job-view/details/summary/LogItem.jsx
  • ui/job-view/details/summary/StatusPanel.jsx
  • ui/job-view/details/summary/SummaryPanel.jsx
  • ui/job-view/details/tabs/AnnotationsTab.jsx
  • ui/job-view/details/tabs/SimilarJobsTab.jsx
  • ui/job-view/details/tabs/TabsPanel.jsx
  • ui/job-view/details/tabs/failureSummary/BugListItem.jsx
  • ui/job-view/details/tabs/failureSummary/FailureSummaryTab.jsx
  • ui/job-view/details/tabs/failureSummary/SuggestionsListItem.jsx
  • ui/job-view/headerbars/ActiveFilters.jsx
  • ui/job-view/headerbars/FiltersMenu.jsx
  • ui/job-view/headerbars/SecondaryNavBar.jsx
  • ui/job-view/headerbars/TierIndicator.jsx
  • ui/job-view/headerbars/TiersMenu.jsx
  • ui/job-view/pushes/JobButton.jsx
  • ui/job-view/pushes/JobGroup.jsx
  • ui/job-view/pushes/Platform.jsx
  • ui/job-view/pushes/Push.jsx
  • ui/job-view/pushes/PushHeader.jsx
  • ui/job-view/pushes/PushLoadErrors.jsx
  • ui/shared/JobInfo.jsx
  • ui/shared/Revision.jsx
  • ui/shared/RevisionList.jsx

Wow, that IS a lot. :) One PR will be fine, yes. It's a lot, true. But that's ok. Please either assign me to the PR when ready, or need-info me here again and I'll review it. Thanks!!

Flags: needinfo?(cdawson)

I opened a PR for this.
I couldn't assign you to the PR, so am using the need-info as an alternative. Thanks!

Flags: needinfo?(cdawson)

Requested some changes in my review

Flags: needinfo?(cdawson)

I assume this is done

Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: