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

Add support for s390x #1142

Merged
merged 23 commits into from
Oct 5, 2023
Merged

Add support for s390x #1142

merged 23 commits into from
Oct 5, 2023

Conversation

dilipgb
Copy link
Contributor

@dilipgb dilipgb commented Jul 5, 2023

This is the PR to enable buildpacks on the s390x platform.

@codecov
Copy link

codecov bot commented Jul 31, 2023

Codecov Report

Merging #1142 (23070ba) into main (105808c) will decrease coverage by 0.52%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1142      +/-   ##
==========================================
- Coverage   65.36%   64.84%   -0.52%     
==========================================
  Files          97       93       -4     
  Lines        6947     6901      -46     
==========================================
- Hits         4540     4474      -66     
- Misses       1992     2008      +16     
- Partials      415      419       +4     
Flag Coverage Δ
os_linux 64.84% <ø> (ø)
os_windows ?
unit 64.84% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@natalieparellano natalieparellano added this to the lifecycle 0.18.0 milestone Aug 3, 2023
@natalieparellano
Copy link
Member

Fixes #811

@dilipgb dilipgb closed this Aug 22, 2023
@dilipgb dilipgb reopened this Aug 22, 2023
@dilipgb dilipgb closed this Aug 22, 2023
@dilipgb dilipgb reopened this Aug 22, 2023
@dilipgb dilipgb closed this Sep 1, 2023
@dilipgb dilipgb reopened this Sep 1, 2023
@dilipgb dilipgb closed this Sep 1, 2023
@dilipgb dilipgb reopened this Sep 1, 2023
Copy link
Contributor

@jabrown85 jabrown85 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great to me! I want @natalieparellano to take a peek as well since this is a bit different than our average PR contribution with all the IBM cloud work happening.

.github/workflows/build.yml Outdated Show resolved Hide resolved
.github/workflows/build.yml Outdated Show resolved Hide resolved
@dilipgb dilipgb marked this pull request as ready for review September 1, 2023 16:07
@dilipgb dilipgb requested a review from a team as a code owner September 1, 2023 16:07
dilipgb and others added 14 commits September 29, 2023 11:52
Signed-off-by: Dilip Gowda Bhagavan <dilip.bhagavan@ibm.com>
Signed-off-by: Dilip Gowda Bhagavan <dilip.bhagavan@ibm.com>
Signed-off-by: Dilip Gowda Bhagavan <dilip.bhagavan@ibm.com>
Signed-off-by: Dilip Gowda Bhagavan <dilip.bhagavan@ibm.com>
Signed-off-by: Dilip Gowda Bhagavan <dilip.bhagavan@ibm.com>
Signed-off-by: Dilip Gowda Bhagavan <dilip.bhagavan@ibm.com>
Signed-off-by: Dilip Gowda Bhagavan <dilip.bhagavan@ibm.com>
Signed-off-by: Dilip Gowda Bhagavan <dilip.bhagavan@ibm.com>
Signed-off-by: Dilip Gowda Bhagavan <dilip.bhagavan@ibm.com>
Signed-off-by: Dilip Gowda Bhagavan <dilip.bhagavan@ibm.com>
Signed-off-by: Dilip Gowda Bhagavan <dilip.bhagavan@ibm.com>
Signed-off-by: Dilip Gowda Bhagavan <dilip.bhagavan@ibm.com>
Signed-off-by: Dilip Gowda Bhagavan <110233170+dilipgb@users.noreply.github.com>
dilipgb and others added 3 commits September 29, 2023 15:47
Signed-off-by: Dilip Gowda Bhagavan <dilip.bhagavan@ibm.com>
Signed-off-by: Dilip Gowda Bhagavan <dilip.bhagavan@ibm.com>
seperated s390x job to different file
Copy link
Member

@natalieparellano natalieparellano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good @dilipgb! I added a couple comments. @jabrown85 do you want to take another look as well?

@jabrown85
Copy link
Contributor

This looks good to me @natalieparellano

dilipgb and others added 6 commits October 4, 2023 13:48
Signed-off-by: Dilip Gowda Bhagavan <dilip.bhagavan@ibm.com>
Signed-off-by: Dilip Gowda Bhagavan <dilip.bhagavan@ibm.com>
Signed-off-by: Dilip Gowda Bhagavan <dilip.bhagavan@ibm.com>
Signed-off-by: Dilip Gowda Bhagavan <dilip.bhagavan@ibm.com>
correcting alignment in build.yml
@natalieparellano natalieparellano merged commit 292aa49 into buildpacks:main Oct 5, 2023
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants