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

Fix issue w/ existing files not being updated in imfs #119

Merged
merged 2 commits into from
Feb 1, 2019

Conversation

ZoteTheMighty
Copy link
Contributor

@ZoteTheMighty ZoteTheMighty commented Feb 1, 2019

Fixes an issue with some recent refactors that caused the imfs to short circuit out of updating the contents of any particular file it already knew about. Only changes higher in the file tree would actually trigger the update.

To do:

  • Add regression test

@coveralls
Copy link

coveralls commented Feb 1, 2019

Coverage Status

Coverage remained the same at 80.952% when pulling 911d965 on ZoteTheMighty:fix-imfs-not-updating into b58fed1 on LPGhatguy:master.

@LPGhatguy LPGhatguy merged commit 0ff59ec into rojo-rbx:master Feb 1, 2019
Dekkonot pushed a commit to UpliftGames/rojo that referenced this pull request Jan 11, 2024
* Fix issue w/ existing files not being updated in imfs

* Add a test for updating files
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