Skip to content

Conversation

@grahamalama
Copy link
Contributor

Pull Request Title

Checklist:

Before submitting your PR, please confirm that you have done the following:

  • I have opened my PR against the staging branch, NOT against main
  • I've run the relevant formatting and linting tools listed in the setup docs
    • I manually ran both docker compose run --rm linter and docker compose run --rm formatter, and both passed
  • I have commented hard-to-understand areas in my code
  • I've reviewed any merge conflicts to make sure they are resolved
  • My changes generate no new warnings
    • as far as I can tell, this is true

Description

We previously had a dependency awkde vendored in this repo. We did this because when we first added it to the project, we were unable to install it with a normal installation method, so we resorted to vendoring and building manually. Now, it appears that we are able to install directly from Github with pipenv, so in this PR we unvendor it.

Type of change

  • Bug fix
  • New feature
    • not a "feature", but it's not a bug or breaking change either
  • Breaking change

How Has This Been Tested?

I wasn't able to test this locally since I'm still working on getting my development environment set up, but the containers build with this change, and I'm hoping CI runs will take care of testing the rest

@vercel
Copy link

vercel bot commented Apr 17, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
vacant-lots-proj ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 26, 2025 9:42pm

@adamzev adamzev merged commit 849a348 into CodeForPhilly:staging Apr 26, 2025
10 checks passed
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.

2 participants