added flag -c contributors logging #35

DaniilLutfulin · 2024-03-15T14:28:19Z

No description provided.

zmm · 2024-04-11T13:31:28Z

Попробовал экспорт, в таблице оказываются не все пользователи. Дал вам админские права на репо - посмотрите вот тут на список участников https://github.com/OSLL/github_repo_commitment_calc/settings/access?query= и попробуйте разобратся, почему не все они оказываются у нас в итоговом файле

thehighestmath · 2024-07-03T09:38:18Z

main.py

@@ -71,8 +72,11 @@ def main():
            git_logger.log_issues(client, repositories, csv_name, token, start, finish)
            if (args.e):
                export_sheets.write_data_to_table(csv_name, args.google_token, args.table_id, args.sheet_id)
-        if args.invites:
-            git_logger.log_invitations(client, repositories, csv_name)
+        if (args.c):


скобки не нужны тут

if args.c:

thehighestmath · 2024-07-03T09:38:25Z

main.py

-            git_logger.log_invitations(client, repositories, csv_name)
+        if (args.c):
+            git_logger.log_contributors(client, repositories, csv_name)
+            if (args.e):


скобки не нужны тут

if args.e:

thehighestmath · 2024-07-03T09:40:00Z

git_logger.py

+        writer.writerow(info)
+
+
+def log_repository_contributors(repository: Repository, csv_name):


тайп хинты это здорово
может их указывать для всех аргументов функций, а не только для некоторых?)

thehighestmath · 2024-07-03T09:42:05Z

git_logger.py

+            'node_id': contributor.node_id,
+            'type': contributor.type,
+            'bio': EMPTY_FIELD if contributor.bio is None else contributor.bio,
+            'site_admin': contributor.site_admin


лучше в конце ставить запятую

объясняю почему

было так

a = [ 1, 2, 3 ]

стало так

a = [ 1, 2, 3, 4 ]

по диффу изменилось 2 строки. 1 добавилась, а другая изменилась, хотя мы только добавили строку.

преждевременная запятая способствует более чистому диффу)

thehighestmath · 2024-07-03T09:42:14Z

git_logger.py

+                'node_id',
+                'type',
+                'bio',
+                'site_admin'


про запятую подумайте :)

thehighestmath · 2024-07-03T09:43:05Z

git_logger.py

@@ -488,3 +488,59 @@ def log_commits(client: Github, repositories, csv_name, start, finish, branch):
            sleep(timedelta)
        except Exception as e:
            print(e)
+
+def log_contributors_to_csv(info, csv_name):


может стоит добавить тайп хинты?)

thehighestmath · 2024-07-03T09:45:03Z

git_logger.py

+        info_tmp = {
+            'repository name': repository.full_name,
+            'login': contributor.login,
+            'name': EMPTY_FIELD if contributor.name is None else contributor.name,


в оракле есть функция nvl https://oracleplsql.ru/nvl-function.html
кажется тут она напрашивается, чтобы вызвать её столько раз сколько нужно, а то для каждого nullable поля писать такой тернарный оператор долго :)

thehighestmath · 2024-07-03T09:45:43Z

git_logger.py

+        }
+        log_contributors_to_csv(info_tmp, csv_name)
+        print(info_tmp)
+        sleep(timedelta)


это из-за ограничений апи нужно?

+посмотрел по коду и понял, что это константа, может стоит капсом назвать её?

thehighestmath · 2024-07-03T09:46:46Z

git_logger.py

+            )
+        )
+    for repo in get_next_repo(client, repositories):
+


удалить?

thehighestmath · 2024-07-03T09:48:44Z

git_logger.py

+        try:
+            log_repository_contributors(repo, csv_name)
+            sleep(timedelta)
+        except Exception as e:


а нам точно все исключение нужно перехватывать?

thehighestmath · 2024-07-05T08:24:18Z

README.md

в ридми нет инфы про то, как должен выглядеть список репозиториев
я по дурости сунул туда ссылки на репозитории
а надо не ссылки

так надо thehighestmath/python-code-check-system
а так не надо https://github.com/thehighestmath/python-code-check-system

можете, пожалуйста, добавить это в ридми?

thehighestmath · 2024-07-05T08:26:27Z

+прошу Вас добавить .gitignore, чтобы случайный git add . не добавил pycache

thehighestmath · 2024-07-05T08:31:39Z

feature-request, хотя это можно в отдельной задаче сделать
а можно ещё добавить количество коммитов, а то смотрю на результат и не совсем понятно насколько большой вклад каждого участника

repository name                         ,login           ,name                ,email       ,url                                ,permissions ,id       ,node_id              ,type ,bio         ,site_admin
thehighestmath/python-code-check-system ,thehighestmath  ,Kirill Kryzhanovsky ,Empty field ,https://github.com/thehighestmath  ,admin       ,42893255 ,MDQ6VXNlcjQyODkzMjU1 ,User ,Empty field ,False
thehighestmath/python-code-check-system ,ArOb-1          ,Artem_dev           ,Empty field ,https://github.com/ArOb-1          ,write       ,75028771 ,MDQ6VXNlcjc1MDI4Nzcx ,User ,Empty field ,False
thehighestmath/python-code-check-system ,GarryKoker      ,Egor                ,Empty field ,https://github.com/GarryKoker      ,write       ,59351926 ,MDQ6VXNlcjU5MzUxOTI2 ,User ,Empty field ,False
thehighestmath/python-code-check-system ,dependabot[bot] ,Empty field         ,Empty field ,https://github.com/apps/dependabot ,none        ,49699333 ,MDM6Qm90NDk2OTkzMzM= ,Bot  ,Empty field ,False

thehighestmath · 2024-07-09T07:00:41Z

@DaniilLutfulin

DaniilLutfulin added 2 commits March 15, 2024 17:16

added_contributors_logging

3c70cc8

added_info_to_readme

21c15af

zmm added this to the 0.1 milestone Mar 26, 2024

zmm added good first issue Good for newcomers feature labels Mar 26, 2024

DaniilLutfulin added 2 commits April 10, 2024 01:28

Added name, fixed permissions

40ba8f5

Merge branch 'master' into print_contributors

a315196

thehighestmath requested changes Jul 3, 2024

View reviewed changes

thehighestmath reviewed Jul 3, 2024

View reviewed changes

thehighestmath reviewed Jul 5, 2024

View reviewed changes

thehighestmath linked an issue Jul 5, 2024 that may be closed by this pull request

Режим -c - вывод списка контрибьюторов #29

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

added flag -c contributors logging #35

added flag -c contributors logging #35

DaniilLutfulin commented Mar 15, 2024

zmm commented Apr 11, 2024

thehighestmath Jul 3, 2024

thehighestmath Jul 3, 2024

thehighestmath Jul 3, 2024

thehighestmath Jul 3, 2024

thehighestmath Jul 3, 2024

thehighestmath Jul 3, 2024

thehighestmath Jul 3, 2024

thehighestmath Jul 3, 2024

thehighestmath Jul 3, 2024

thehighestmath Jul 3, 2024

thehighestmath Jul 3, 2024

thehighestmath Jul 5, 2024

thehighestmath commented Jul 5, 2024

thehighestmath commented Jul 5, 2024 •

edited

Loading

thehighestmath commented Jul 9, 2024

		writer.writerow(info)


		def log_repository_contributors(repository: Repository, csv_name):

added flag -c contributors logging #35

Are you sure you want to change the base?

added flag -c contributors logging #35

Conversation

DaniilLutfulin commented Mar 15, 2024

zmm commented Apr 11, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

thehighestmath commented Jul 5, 2024

thehighestmath commented Jul 5, 2024 • edited Loading

thehighestmath commented Jul 9, 2024

thehighestmath commented Jul 5, 2024 •

edited

Loading