Tim Jenness

Am I correct in thinking that nothing calls `_svnRevision` anyhow?

Nothing in the source references that function. It can be removed.

Sounds interesting. Can you make a pull request please? (and preferably a Jira ticket branch)

This PR is 4 years old now but it doesn't look like we have converged. Should it remain open?

Should we close this PR without merging?

A tuple seems right to me given that you don't want someone to change the content, but there is a problem that the Python docs themselves use a list: https://docs.python.org/3/tutorial/modules.html#importing-from-a-package...

The other downside of tuple is that: ```python __all__ = ("myfunc") ``` is not what you want but ```python __all__ = ["myfunc"] ``` will work without having to worry about...

I'm happy to let @ktlim decide.

@yalsayyad I've rebased and dealt with the second half of @jonathansick 's comments.