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

Flag to reload sources unconditionally #67

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dm3
Copy link

@dm3 dm3 commented Mar 28, 2016

This is sometimes useful - e.g. recently ran into something which produced a compilation warning but didn't have any effect on the functionality. Not sure if it's worth including as a core task option, but having something to force reload would be great to have!

@Deraen
Copy link
Contributor

Deraen commented Jun 13, 2016

While this is very simple change, I'd rather not merge this unless there are good reason why this is needed.

Often the warnings affect functionality and they should always be fixed.

@dm3
Copy link
Author

dm3 commented Jun 13, 2016

One such thing is the Javelin cell= macro that deep walks the code and hoists the variables outside of the function. It has some obscure bug that generates warnings when destructuring is used inside the macro. The warnings do not affect functionality in any way.

That's the only case that I'm aware of so far. I find it very annoying if there's no way to make the tools do what you want. I'm OK if there's another way to achieve the same if a task option is too intrusive, e.g. env option?

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