Project

General

Profile

Refactor #35890

Enable harmony mode with Uglifier to use ES6 syntax

Added by Nofar Alfassi 3 months ago. Updated 2 months ago.

Status:
Rejected
Priority:
Normal
Assignee:
Nofar Alfassi
Category:
-
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:
Pull request:
https://github.com/theforeman/foreman/pull/9594, https://github.com/theforeman/foreman/pull/9567
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

Using ES6 syntax requires enabling harmony mode with Uglifier.

Associated revisions

Revision d590d565 (diff)
Added by Nofar Alfassi 2 months ago

Fixes #35890 - enable harmony mode with Uglifier to use ES6 syntax

Added some ES6 features across multiple pages to show that everything works

Revision 0106d6c7 (diff)
Added by Ron Lavi 2 months ago

Refs #35890 - revert enable harmony mode with Uglifier to use ES6 syntax

This reverts commit d590d56578e3d70972efe5138558fd9007140a93.

History

#1 Updated by The Foreman Bot 3 months ago

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

#2 Updated by The Foreman Bot 2 months ago

  • Fixed in Releases 3.6.0 added

#3 Updated by Nofar Alfassi 2 months ago

  • Status changed from Ready For Testing to Closed

#4 Updated by The Foreman Bot 2 months ago

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

#5 Updated by Evgeni Golov 2 months ago

  • Status changed from Closed to New

#6 Updated by Evgeni Golov 2 months ago

  • Fixed in Releases deleted (3.6.0)

#7 Updated by Nofar Alfassi 2 months ago

  • Status changed from New to Closed

Closing for now.
This change failed in nightly: https://community.theforeman.org/t/foreman-nightly-rpm-pipeline-1763-failed/31996.
We decided to skip this change since we don't need it for now.

#8 Updated by Ewoud Kohl van Wijngaarden 2 months ago

  • Status changed from Closed to Rejected

Also available in: Atom PDF