Skip to content
This repository has been archived by the owner on Sep 26, 2024. It is now read-only.

Add utils to measure portal's uptime #71

Open
wants to merge 21 commits into
base: main
Choose a base branch
from
Open

Add utils to measure portal's uptime #71

wants to merge 21 commits into from

Conversation

bcbernardo
Copy link
Collaborator

Should close #41

@bcbernardo bcbernardo added the analysis Issues related to the analysis of official gazettes portals label Feb 17, 2021

Os utilitários contidos nesse sub-repositório podem ser instalados como pacotes
Python avulsos. Para isso, você deve ter instalada na sua máquina uma versão
Python compatível (3.7 ou superior).
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see you use typing.Final in the code, but typing.Final was introduced in Python 3.8. There's also a Python 3.7y reference in setup.cfg.

@@ -0,0 +1,320 @@
# Guia de validação
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this file be named VALIDATION-GUIDE.md? I guess VALIDATION might be a typo.

@@ -0,0 +1,36 @@
# Environment variables

### IMPORTANT: when done editing this file, rename it to ".env"
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You might want to take a look in Createnv to easily automate that.

Disclaimer: I am the creator of that package! But, I mean… I wrote it exactly to make it easy to onboard newcomers to projects using .env files ✨

@@ -0,0 +1,20 @@
# Copyright 2020 Open Knowledge Brasil
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe 2021?

Comment on lines +89 to +90
from kaggle.api.kaggle_api_extended import KaggleApi # type: ignore
from kaggle.models.kaggle_models_extended import Metadata # type: ignore
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

# type: ignore tells linters to ignore these lines, but tells nothing to fellow developers about why you are importing this here (and not in the top of the file as usual). What about a friendly comment here to help fellow coders?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
analysis Issues related to the analysis of official gazettes portals
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Medir o tempo em serviço dos portais // Measure portals' uptime
2 participants