Skip to content

Conversation

@Fryguy
Copy link
Member

@Fryguy Fryguy commented Sep 21, 2020

@himdel Please review. We had sorted it originally, but when we switched to generating the packages only it was irrelevant. So when we added back the modules list it was unsorted.

@miq-bot
Copy link
Member

miq-bot commented Sep 21, 2020

Checked commit Fryguy@5581af0 with ruby 2.6.3, rubocop 0.69.0, haml-lint 0.28.0, and yamllint
0 files checked, 0 offenses detected
Everything looks fine. 🍰

@himdel himdel merged commit bee2cdc into ManageIQ:master Sep 23, 2020
@Fryguy Fryguy deleted the sort_modules_manifest branch September 23, 2020 16:31
simaishi pushed a commit that referenced this pull request Sep 24, 2020
Sort the webpack modules for consistent output

(cherry picked from commit bee2cdc)
@simaishi
Copy link
Contributor

Jansa backport details:

$ git log -1
commit 80e81ab2c70ce1dcce4d6febf98b47bbd706c37f
Author: Martin Hradil <mhradil@redhat.com>
Date:   Wed Sep 23 13:20:31 2020 +0000

    Merge pull request #7332 from Fryguy/sort_modules_manifest

    Sort the webpack modules for consistent output

    (cherry picked from commit bee2cdc4ef5468495d390903905dd25c0791df9d)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants