-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Remove useless col_name_map arg from internal table functions #16364
Remove useless col_name_map arg from internal table functions #16364
Conversation
Thank you for your contribution to Astropy! 🌌 This checklist is meant to remind the package maintainers who will review this pull request of some common things to look for.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Agreed with @pllim that for changes in private functions a changelog entry is not needed (though if you really want it, I don't mind; but then it should be api
, not feature
)
Somehow there is a merge conflict now... |
Huh weird, there is conflict. You might need to rebase and also fix the pre-commit check. |
13342fe
to
12cf1fd
Compare
Rebased and hopefully squared away now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like mhvk is happy. Diff also LGTM. Thanks!
Description
The
col_name_map
argument passed around in the internal_join
,_hstack
and_vstack
functions can be useful for diagnostics. However they are not exposed in the public API so they are not doing anything useful. I don't recall if things used to be different but at this point it seems they can just be cut.