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

Create test-specific 🐳 file #1082

Merged
merged 36 commits into from
May 4, 2024
Merged

Create test-specific 🐳 file #1082

merged 36 commits into from
May 4, 2024

Conversation

michplunkett
Copy link
Collaborator

@michplunkett michplunkett commented Dec 30, 2023

Related to:

Description of Changes

Creating a test-specific 🐳 Docker file. This will be the first step in implementing PostgreSQL in the testing environment so that we can get a true 1:1 comparison between the environments.

Tests and linting

  • This branch is up-to-date with the develop branch.
  • pytest passes on my local development environment.
  • pre-commit passes on my local development environment.

@michplunkett michplunkett self-assigned this Dec 30, 2023
COPY package.json /usr/src/app/
RUN yarn

COPY test_data.py /usr/src/app/
COPY mypy.ini /usr/src/app/
EXPOSE 3000
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This line was redundant.

on:
pull_request:
branches:
- develop
- main
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We have stopped using main, so I removed it from this file.

@michplunkett michplunkett marked this pull request as ready for review February 12, 2024 20:57
@michplunkett michplunkett mentioned this pull request Apr 30, 2024
4 tasks
@b-meson
Copy link
Member

b-meson commented May 3, 2024

Thanks for working on this @michplunkett, feel free to merge it whenever you can.

@michplunkett
Copy link
Collaborator Author

I'm workin' on some school stuff now, but I'll likely merge it tomorrow when I can make sure everything is 💯 .

@michplunkett
Copy link
Collaborator Author

I ran everything on my local again, and it looks like we're good to go.

@michplunkett michplunkett merged commit 2d82e06 into develop May 4, 2024
3 checks passed
@michplunkett michplunkett deleted the test_docker_file branch May 4, 2024 17:44
@michplunkett michplunkett mentioned this pull request Jun 2, 2024
3 tasks
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.

2 participants