Project

General

Profile

Feature #25217

Update the eslint configuration to use eslint-plugin-patternfly-react

Added by Avi Sharvit about 1 year ago. Updated 11 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
JavaScript stack
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:
Pull request:
https://github.com/theforeman/foreman/pull/6371, https://github.com/theforeman/foreman/pull/6370, https://github.com/theforeman/foreman/pull/6369, https://github.com/theforeman/foreman/pull/6276, https://github.com/theforeman/foreman/pull/6275, https://github.com/theforeman/foreman/pull/6274, https://github.com/theforeman/foreman/pull/6273, https://github.com/theforeman/foreman/pull/6250, https://github.com/theforeman/foreman/pull/6248, https://github.com/theforeman/foreman/pull/6247, https://github.com/theforeman/foreman/pull/6242, https://github.com/theforeman/foreman/pull/6241, https://github.com/theforeman/foreman/pull/6232, https://github.com/theforeman/foreman/pull/6231, https://github.com/theforeman/foreman/pull/6230, https://github.com/theforeman/foreman/pull/6229, https://github.com/theforeman/foreman/pull/6228, https://github.com/theforeman/foreman/pull/6227, https://github.com/theforeman/foreman/pull/6226, https://github.com/theforeman/foreman/pull/6225, https://github.com/theforeman/foreman/pull/6224, https://github.com/theforeman/foreman/pull/6223, https://github.com/theforeman/foreman/pull/6221, https://github.com/theforeman/foreman/pull/6219, https://github.com/theforeman/foreman/pull/6217, https://github.com/theforeman/foreman/pull/6216, https://github.com/theforeman/foreman/pull/6212, https://github.com/theforeman/foreman/pull/6374, https://github.com/theforeman/foreman/pull/6205, https://github.com/theforeman/foreman/pull/6278, https://github.com/theforeman/foreman/pull/6277, https://github.com/theforeman/foreman/pull/6148, https://github.com/theforeman/foreman/pull/6206, https://github.com/theforeman/foreman/pull/6373, https://github.com/theforeman/foreman/pull/6372
Fixed in Releases:
Found in Releases:

Related issues

Related to Foreman - Bug #26286: Audits page displays warning when some audits are lacking user detailsClosed

Associated revisions

Revision 44cc19ff (diff)
Added by Avi Sharvit about 1 year ago

Refs #25217 - Lint fix - use camelcase

Revision 9bdbf794 (diff)
Added by Avi Sharvit about 1 year ago

Refs #25217 - Add prop-types to AuditList

Revision a74a39a2 (diff)
Added by Avi Sharvit about 1 year ago

Refs #25217 - Add missing default-prop to BarChart

Revision 88d4cc71 (diff)
Added by Avi Sharvit about 1 year ago

Refs #25217 - Add prop-types to SearchInput

Revision 259c5baa (diff)
Added by Avi Sharvit about 1 year ago

Refs #25217 - Add prop-types to MessageBox

Revision eef4e92f (diff)
Added by Avi Sharvit about 1 year ago

Refs #25217 - Add prop-types to Loader

Revision 3fa5a308 (diff)
Added by Avi Sharvit about 1 year ago

Refs #25217 - Add prop-types to Icon

Revision 2ffcd59e (diff)
Added by Avi Sharvit about 1 year ago

Refs #25217 - Add default-props DocumentationLink

Revision 155a9c0c (diff)
Added by Avi Sharvit about 1 year ago

Refs #25217 - Remove .js from js imports

Revision 7945b9da (diff)
Added by Avi Sharvit about 1 year ago

Refs #25217 - Refactor boorkamrks, add prop-types

Revision 3bb22380 (diff)
Added by Avi Sharvit about 1 year ago

Refs #25217 - Add default-props to Alert

Revision 581729cd (diff)
Added by Avi Sharvit about 1 year ago

Refs #25217 - Convert password-strength be dummy

Revision bebf4689 (diff)
Added by Avi Sharvit about 1 year ago

Refs #25217 - Fix small issue with the Layout

Revision af7d009f (diff)
Added by Avi Sharvit about 1 year ago

Refs #25217 - Add prop-types to the BreadcrumbBar

Revision ccc2eb42 (diff)
Added by Avi Sharvit about 1 year ago

Refs #25217 - Add default-props to FactChart

Revision 75501405 (diff)
Added by Avi Sharvit about 1 year ago

Refs #25217 - Fix target="_blank" vulnerability

Revision 01c6dc1e (diff)
Added by Avi Sharvit about 1 year ago

Refs #25217 - ConfigReport data should be required

Update the ConfigReport prop-types so `data` is a required prop

Revision c0eb6b3f (diff)
Added by Avi Sharvit about 1 year ago

Refs #25217 - Add alt to breadcrumb icon

To support better a11y

Revision cd0e17c3 (diff)
Added by Avi Sharvit about 1 year ago

Refs #25217 - Add prop-types to forms

Revision e4176c54 (diff)
Added by Avi Sharvit about 1 year ago

Refs #25217 - Adds prop-types to host/power-status

Revision 909556e8 (diff)
Added by Avi Sharvit about 1 year ago

Refs #25217 - Refactor hosts/storage prop-types

Revision e3eb262b (diff)
Added by Avi Sharvit about 1 year ago

Refs #25217 - Refactor hosts/vmware/disk component

Adds prop-types, default-props and fix some code styling

Revision ccbd5a77 (diff)
Added by Avi Sharvit 12 months ago

Refs #25217 - Refactor diff-modal prop-types

Revision 251e31e1 (diff)
Added by Avi Sharvit 12 months ago

Refs #25217 - Refactor pagination prop-types

Revision ace8dfeb (diff)
Added by Avi Sharvit 12 months ago

Refs #25217 - Refactor statistics prop-types

Revision 6a9879ca (diff)
Added by Avi Sharvit 12 months ago

Refs #25217 - Refactor Markdown prop-types

  • Adds default-props
  • Convert to functional component

Revision ba99da20 (diff)
Added by Avi Sharvit 12 months ago

Refs #25217 - Refactor notifications prop-types

  • Adds snapshot testing for notifications toggle-icon

Revision df6f975a (diff)
Added by Avi Sharvit 12 months ago

Refs #25217 - Refactor ToastNotifications prop-types

Revision 839bcbbd (diff)
Added by Avi Sharvit 12 months ago

Fixes #25217 - Install patternfly-react eslint

Update the eslint configuration to use eslint-plugin-patternfly-react

Revision e43dae73 (diff)
Added by Avi Sharvit 12 months ago

Refs #25217 - Automatic eslint fix

Run `npm run lint -- --fix`
to automatically fix all the fixable linting issues

Revision c18d6746 (diff)
Added by Avi Sharvit 12 months ago

Refs #25217 - Fix all linting issues manually

Revision f436c6ca (diff)
Added by Avi Sharvit 11 months ago

Refs #25217 - fix charts eslint

Revision 8bdbbd5b (diff)
Added by Avi Sharvit 11 months ago

Refs #25217 - fix forms eslint

Revision 3f6e13eb (diff)
Added by Avi Sharvit 11 months ago

Refs #25217 - fix Layout eslint

Revision 64515c69 (diff)
Added by Avi Sharvit 11 months ago

Refs #25217 - fix search-bar eslint

Revision 065bdddc (diff)
Added by Avi Sharvit 11 months ago

Refs #25217 - fix hosts eslint

Revision 395cd65e (diff)
Added by Avi Sharvit 11 months ago

Refs #25217 - fix AutoComplete eslint

History

#1 Updated by The Foreman Bot about 1 year ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/6148 added

#2 Updated by The Foreman Bot about 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/6205 added

#3 Updated by The Foreman Bot about 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/6206 added

#4 Updated by The Foreman Bot about 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/6212 added

#5 Updated by The Foreman Bot about 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/6216 added

#6 Updated by The Foreman Bot about 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/6217 added

#7 Updated by The Foreman Bot about 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/6219 added

#8 Updated by The Foreman Bot about 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/6221 added

#9 Updated by The Foreman Bot about 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/6223 added

#10 Updated by The Foreman Bot about 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/6224 added

#11 Updated by The Foreman Bot about 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/6225 added

#12 Updated by The Foreman Bot about 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/6226 added

#13 Updated by The Foreman Bot about 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/6227 added

#14 Updated by The Foreman Bot about 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/6228 added

#15 Updated by The Foreman Bot about 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/6229 added

#16 Updated by The Foreman Bot about 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/6230 added

#17 Updated by The Foreman Bot about 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/6231 added

#18 Updated by The Foreman Bot about 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/6232 added

#19 Updated by The Foreman Bot about 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/6241 added

#20 Updated by The Foreman Bot about 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/6242 added

#21 Updated by The Foreman Bot about 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/6247 added

#22 Updated by The Foreman Bot about 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/6248 added

#23 Updated by The Foreman Bot about 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/6250 added

#24 Updated by The Foreman Bot 12 months ago

  • Pull request https://github.com/theforeman/foreman/pull/6273 added

#25 Updated by The Foreman Bot 12 months ago

  • Pull request https://github.com/theforeman/foreman/pull/6274 added

#26 Updated by The Foreman Bot 12 months ago

  • Pull request https://github.com/theforeman/foreman/pull/6275 added

#27 Updated by The Foreman Bot 12 months ago

  • Pull request https://github.com/theforeman/foreman/pull/6276 added

#28 Updated by The Foreman Bot 12 months ago

  • Pull request https://github.com/theforeman/foreman/pull/6277 added

#29 Updated by The Foreman Bot 12 months ago

  • Pull request https://github.com/theforeman/foreman/pull/6278 added

#30 Updated by Avi Sharvit 12 months ago

  • Status changed from Ready For Testing to Closed

#31 Updated by Tomer Brisker 12 months ago

  • Fixed in Releases 1.21.0 added

Note: in the future, please try to use one pr per issue - otherwise it is hard to tell if all prs have been merged or not.

#32 Updated by The Foreman Bot 11 months ago

  • Pull request https://github.com/theforeman/foreman/pull/6369 added

#33 Updated by The Foreman Bot 11 months ago

  • Pull request https://github.com/theforeman/foreman/pull/6370 added

#34 Updated by The Foreman Bot 11 months ago

  • Pull request https://github.com/theforeman/foreman/pull/6371 added

#35 Updated by The Foreman Bot 11 months ago

  • Pull request https://github.com/theforeman/foreman/pull/6372 added

#36 Updated by The Foreman Bot 11 months ago

  • Pull request https://github.com/theforeman/foreman/pull/6373 added

#37 Updated by The Foreman Bot 11 months ago

  • Pull request https://github.com/theforeman/foreman/pull/6374 added

#38 Updated by Tomer Brisker 8 months ago

  • Related to Bug #26286: Audits page displays warning when some audits are lacking user details added

Also available in: Atom PDF