Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lock down es-abstract to avoid build issues on s390x #1857

Merged
merged 2 commits into from
Aug 1, 2023

Conversation

bdunne
Copy link
Member

@bdunne bdunne commented Aug 1, 2023

bdunne added a commit to bdunne/manageiq-rpm_build that referenced this pull request Aug 1, 2023
Yarn 1 needs to rewrite the lockfile
es-abstract is locked down by ManageIQ/manageiq-ui-service#1857
@miq-bot
Copy link
Member

miq-bot commented Aug 1, 2023

Checked commits bdunne/manageiq-ui-service@cf484a2~...710ecc4 with ruby 2.6.10, rubocop 1.28.2, haml-lint 0.35.0, and yamllint
0 files checked, 0 offenses detected
Everything looks fine. 🍰

@Fryguy Fryguy merged commit cab1c3e into ManageIQ:master Aug 1, 2023
1 of 2 checks passed
@bdunne bdunne deleted the es-abstract branch August 1, 2023 21:03
@Fryguy
Copy link
Member

Fryguy commented Aug 2, 2023

Backported to petrosian in commit c74b698.

commit c74b698c3f486221045adb587911d4a79f860a62
Author: Jason Frey <fryguy9@gmail.com>
Date:   Tue Aug 1 16:59:06 2023 -0400

    Merge pull request #1857 from bdunne/es-abstract
    
    Lock down es-abstract to avoid build issues on s390x
    
    (cherry picked from commit cab1c3e4d55d0fd69e91594500ade08c87d56bf2)

Fryguy added a commit that referenced this pull request Aug 2, 2023
Lock down es-abstract to avoid build issues on s390x

(cherry picked from commit cab1c3e)
Fryguy added a commit to Fryguy/manageiq-ui-service that referenced this pull request Dec 7, 2023
Now that we are on yarn 4.0.2, which can support newer versions of
es-abstract on s390x, this commit reverts all of the lockdowns to
packages for that reason.

This commit reverts cf484a2 (ManageIQ#1857), 95f6c55/68335b56 (ManageIQ#1860), and
d72356f (ManageIQ#1861)
@Fryguy Fryguy mentioned this pull request Dec 7, 2023
Fryguy added a commit to Fryguy/manageiq-ui-service that referenced this pull request Dec 7, 2023
Now that we are on yarn 4.0.2, which can support newer versions of
es-abstract on s390x, this commit reverts all of the lockdowns to
packages for that reason.

This commit reverts cf484a2 (ManageIQ#1857), 95f6c55 / 68335b5 (ManageIQ#1860), and
d72356f (ManageIQ#1861)
Fryguy added a commit to Fryguy/manageiq-ui-service that referenced this pull request Dec 7, 2023
Now that we are on yarn 4.0.2, which can support newer versions of
es-abstract on s390x, this commit reverts all of the lockdowns to
packages for that reason.

This commit reverts cf484a2 (ManageIQ#1857), 95f6c55 / 68335b5 (ManageIQ#1860), and
d72356f (ManageIQ#1861)
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.

3 participants