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

Clearly denote adapter interface for file reloading #24989

Closed
wants to merge 2 commits into from

Conversation

maclover7
Copy link
Contributor

cc @tenderlove

Laying groundwork for #24981.

@matthewd
Copy link
Member

Thanks, but this is a heavyweight adapter pattern for something that's just recently grown a second implementation, and I don't foresee a third.

@matthewd matthewd closed this May 12, 2016
@maclover7
Copy link
Contributor Author

This was closed due to uncertain future about if/how @tenderlove's forking idea would be implemented, and if an adapter/plugin pattern would be used.

@maclover7 maclover7 deleted the jm-file-update-strategy branch May 12, 2016 02:08
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.

None yet

2 participants