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

Migrate from inlining to copying for asset dependencies #102

Merged
merged 14 commits into from
Jun 7, 2021

Conversation

jonathanmayer
Copy link
Contributor

Resolves #101.

@jonathanmayer jonathanmayer added the enhancement New feature or request label Jun 7, 2021
@jonathanmayer jonathanmayer requested a review from rhelmer June 7, 2021 01:07
@jonathanmayer jonathanmayer self-assigned this Jun 7, 2021
Copy link
Contributor

@rhelmer rhelmer left a comment

Choose a reason for hiding this comment

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

this lgtm, I appreciate the little test extension! we could pretty easily turn little one-off test extensions like this into integration tests too.

@jonathanmayer jonathanmayer merged commit fc22ede into main Jun 7, 2021
@jonathanmayer jonathanmayer deleted the inlining-to-copying branch June 7, 2021 19:27
rhelmer added a commit to citp/news-disinformation-study that referenced this pull request Jun 14, 2021
rhelmer added a commit to citp/news-disinformation-study that referenced this pull request Jun 25, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Replace inlined content with copied content
2 participants