feat: create CustomPropertiesStream class

[MindaugasN]

create CustomPropertiesStream class that fetches data from https://docs.github.com/en/rest/repos/custom-properties?apiVersion=2022-11-28

[edgarrmondragon]

Seeing a 404 in the tests: Custom properties only supported for organizations.

Perhaps we should think of a way of skipping/deselecting this stream for non-org namespaces?

[MindaugasN]

Seeing a 404 in the tests: Custom properties only supported for organizations.

Perhaps we should think of a way of skipping/deselecting this stream for non-org namespaces?

Is there already an example with that? And how to tell if a namespace is org / non-org?

[edgarrmondragon]

Seeing a 404 in the tests: Custom properties only supported for organizations.
Perhaps we should think of a way of skipping/deselecting this stream for non-org namespaces?

Is there already an example with that? And how to tell if a namespace is org / non-org?

@MindaugasN I think moving it here

tap-github/tap_github/streams.py

Lines 118 to 121 in f41b66d

	ORGANIZATIONS = (
	{"organizations"},
	[OrganizationStream, TeamMembersStream, TeamRolesStream, TeamsStream],
	)

would be enough

[MindaugasN]

I moved, but first of all, I am not sure, if I did that correctly.
What I see though, puting stream like this (under organizations) - it does not work for me anymore, CustomPropertiesStream should be under repository stream. Maybe you didn't mean that I should move CustomPropertiesStream under organization_streams.py?
Another thing, I don't understand why pre-commit.ci fails. Do you have an idea?

[edgarrmondragon]

I moved, but first of all, I am not sure, if I did that correctly. What I see though, puting stream like this (under organizations) - it does not work for me anymore, CustomPropertiesStream should be under repository stream.

Can you say more about this? What do you mean it's not working anymore?

Maybe you didn't mean that I should move CustomPropertiesStream under organization_streams.py?

Not really. The stream class can stay in tap_github/repository_streams.py.

Another thing, I don't understand why pre-commit.ci fails. Do you have an idea?

I think you're missing a newline at the end of tap_github/organization_streams.py.

[MindaugasN]

Not really. The stream class can stay in tap_github/repository_streams.py.

Got it, that solved all my problems :)

And with the test - I ran Black locally and formatted my code

[edgarrmondragon]

I think we're back at #268 (comment) 😅