Eeek! Bug in the show_all_tags.py code

[PySimpleGUI]

I just discovered that all of my PEP8 bindings that have an _ in them are not in the documentation.

This is because of this line of code that generates the code that goes into the .md file. Here is the version that works:

print('\n'.join([f"## {j[0]}\n\n<!-- <+{i[0]}.{j[0]}+> -->\n" for j in inspect.getmembers(i[1]) if not j[0].startswith('_')  ]))

It used to read:

    print('\n'.join([f"### {j[0]}\n\n<!-- <+{i[0]}.{j[0]}+> -->\n" for j in inspect.getmembers(i[1]) if '_' not in j[0]  ]))

Which filters out all of the methods that have an _ in them anywhere. DOH!

[nngogol]

So, you want to keep all methods, that starts with one "_"? I mean, methods like these:

for me def \_{1}\w+[^\_]\(

def _RightClickMenuCallback
def _MenuItemChosenCallback
def _FindReturnKeyBoundButton
def _TextClickedHandler
def _ReturnKeyHandler
def _ListboxSelectHandler
def _ComboboxSelectHandler
def _RadioHandler
def _CheckboxHandler
def _TabGroupSelectHandler
def _KeyboardHandler
def _ClickHandler
def _SpinChangedHandler
def _MenuItemChosenCallback
def _convert_xy_to_canvas_xy
def _convert_canvas_xy_to_xy
def _GetElementAtLocation
def _GetElementAtLocation
def _GetElementAtLocation
def _SliderChangedHandler
def _on_mousewheel
def _GetElementAtLocation
def __setup_styles
def __place_widgets
def __config_calendar
def __setup_selection
def __minsize
def _build_calendar
def _show_selection
def _pressed
def _prev_month
def _next_month
def _MenuItemChosenCallback
def _Add
def _AddNode
def _NodeStr
def _Show
def _GetElementAtLocation
def _GetDefaultElementSize
def _AutoCloseAlarmCallback
def _TimeoutAlarmCallback
def _ReadNonBlocking
def _BuildKeyDict
def _BuildKeyDictForWindow
def _StartMove
def _StopMove
def _OnMotion
def _KeyboardCallback
def _MouseWheelCallback
def _Close
def _OnClosingCallback
def _callback_main_debugger_window_create_keystroke
def _callback_popout_window_create_keystroke
def _BuildResults
def _BuildResultsForSubform
def _FindElementWithFocusInSubForm
def _GetNumLinesNeeded
def _build_main_debugger_window
def _refresh_main_debugger_window
def _display_all_vars
def _choose_auto_watches
def _build_floating_window
def _refresh_floating_window
def _refresh_debugger

[PySimpleGUI]

No, I don't want names that start with _.

I've fixed the problem myself. You can get the change if you want from my github or at the top of this Issue. I'm not sure if I added a better fix after filing this one.

I thought my description was detailed and accurate. Maybe you can advise me on how to better communicate?

This is the last line which described the problem

Which filters out all of the methods that have an _ in them anywhere. DOH!

The key word is anywhere

[PySimpleGUI]

Do you not understand the bug / fix?

The bug was using this:

if '_' not in j[0]

when this is what I believe was meant:

if not j[0].startswith('_')