Skip to content
This repository has been archived by the owner on Sep 18, 2024. It is now read-only.

Specify docker platform in composer #190

Merged
merged 4 commits into from
Sep 10, 2024
Merged

Specify docker platform in composer #190

merged 4 commits into from
Sep 10, 2024

Conversation

acouch
Copy link
Member

@acouch acouch commented Sep 6, 2024

Summary

Fixes #{ISSUE}

Time to review: 5 mins

Changes proposed

  • specifies platform type for docker compose to avoid local build issues. This matches the build type in the CI.
  • removes unnecessary file
  • adds vim swapfile to .gitignore

Copy link

github-actions bot commented Sep 6, 2024

Coverage report for ./frontend

St.
Category Percentage Covered / Total
🟢 Statements 90.06% 707/785
🟡 Branches 71.33% 204/286
🟢 Functions 86.31% 145/168
🟢 Lines 90.75% 667/735

Test suite run success

155 tests passing in 47 suites.

Report generated by 🧪jest coverage report action from b2d532e

@acouch acouch changed the title Remove uncessary file Specify docker platform in composer Sep 10, 2024
@acouch acouch marked this pull request as ready for review September 10, 2024 15:57
@acouch acouch merged commit b0df4e0 into main Sep 10, 2024
12 checks passed
@acouch acouch deleted the acouch/fe-cleanup branch September 10, 2024 15:57
acouch added a commit that referenced this pull request Sep 18, 2024
acouch added a commit that referenced this pull request Sep 18, 2024
acouch added a commit to HHS/simpler-grants-gov that referenced this pull request Sep 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants