Skip to content
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

Renaming #3

Closed
wants to merge 9 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/ISSUE_TEMPLATE/bug_report.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ Provide a clear and concise description of the bug.
Include source code:

```python
from weatherapi.app import app
from google_sheets.app import app
...
```

Expand Down
2 changes: 1 addition & 1 deletion .github/ISSUE_TEMPLATE/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,4 +5,4 @@ contact_links:
about: Please report security vulnerabilities to [email protected]
- name: Question or Problem
about: Ask a question or ask about a problem in GitHub Discussions.
url: https://github.com/airtai/weatherapi/discussions/categories/questions
url: https://github.com/airtai/google-sheets/discussions/categories/questions
4 changes: 2 additions & 2 deletions .github/ISSUE_TEMPLATE/feature_request.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,10 +15,10 @@ Provide a clear and concise description of the problem you've encountered. For e
Clearly and concisely describe the desired outcome or solution.

**Feature code example**
To help others understand the proposed feature, illustrate it with a **weatherapi** code example:
To help others understand the proposed feature, illustrate it with a **google-sheets** code example:

```python
from weatherapi.app import app
from google_sheets.app import app
...
```

Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/docker_cleanup.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ jobs:
with:
# NOTE: at now only orgs is supported
owner: airtai
name: weatherapi
name: google-sheets

token: ${{ secrets.GITHUB_TOKEN }}
# Keep latest N untagged images
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -29,11 +29,11 @@ jobs:
pip install -e ".[docs,lint]"
- name: Run mypy
shell: bash
run: mypy weatherapi tests
run: mypy google_sheets tests

- name: Run bandit
shell: bash
run: bandit -c pyproject.toml -r weatherapi
run: bandit -c pyproject.toml -r google_sheets

- name: Run Semgrep
shell: bash
Expand Down Expand Up @@ -146,7 +146,7 @@ jobs:
- run: ls -la coverage
- run: coverage combine coverage
- run: coverage report
- run: coverage html --show-contexts --title "weatherapi coverage for ${{ github.sha }}"
- run: coverage html --show-contexts --title "google-sheets coverage for ${{ github.sha }}"

- name: Store coverage html
uses: actions/upload-artifact@v4
Expand Down
10 changes: 5 additions & 5 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -36,21 +36,21 @@ After activating the virtual environment as described above, run:
pip install -e ".[dev]"
```

This will install all the dependencies and your local **WeatherAPI** in your virtual environment.
This will install all the dependencies and your local **GoogleSheetsAPI** in your virtual environment.

### Using Your local **WeatherAPI**
### Using Your local **GoogleSheetsAPI**

If you create a Python file that imports and uses **WeatherAPI**, and run it with the Python from your local environment, it will use your local **WeatherAPI** source code.
If you create a Python file that imports and uses **GoogleSheetsAPI**, and run it with the Python from your local environment, it will use your local **GoogleSheetsAPI** source code.

Whenever you update your local **WeatherAPI** source code, it will automatically use the latest version when you run your Python file again. This is because it is installed with `-e`.
Whenever you update your local **GoogleSheetsAPI** source code, it will automatically use the latest version when you run your Python file again. This is because it is installed with `-e`.

This way, you don't have to "install" your local version to be able to test every change.

## Running Tests

### Pytest

To run tests with your current **WeatherAPI** application and Python environment, use:
To run tests with your current **GoogleSheetsAPI** application and Python environment, use:

```bash
pytest tests
Expand Down
2 changes: 1 addition & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ RUN update-alternatives --install /usr/bin/python3 python3 /usr/bin/python3.11 1

RUN python3 -m pip install --upgrade pip

COPY weatherapi ./weatherapi
COPY google_sheets ./google_sheets
COPY scripts/* pyproject.toml README.md ./
RUN pip install -e ".[dev]"

Expand Down
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
# Weather API
A simple weather API made for usage with FastAgency
# Google Sheets API
A simple Google Sheets API made for usage with FastAgency
6 changes: 3 additions & 3 deletions SECURITY.md
Original file line number Diff line number Diff line change
@@ -1,14 +1,14 @@
# Security Policy

Security and stability are paramount for WeatherAPI.
Security and stability are paramount for google-sheets.

Learn more below. 👇

## Versions

The latest version of WeatherAPI is actively supported.
The latest version of google-sheets is actively supported.

We strongly encourage you to write tests for your application and regularly update your WeatherAPI version after confirming that your tests pass. This ensures you benefit from the latest features, bug fixes, and **security updates**.
We strongly encourage you to write tests for your application and regularly update your google-sheets version after confirming that your tests pass. This ensures you benefit from the latest features, bug fixes, and **security updates**.

## Reporting a Vulnerability

Expand Down
8 changes: 4 additions & 4 deletions docker-compose.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ x-logging: &default-logging

version: "3.4"

name: weatherapi
name: google-sheets
services:
fastapi-app: # nosemgrep
image: ghcr.io/${GITHUB_REPOSITORY}:${TAG}
Expand All @@ -22,8 +22,8 @@ services:
- DOMAIN=${DOMAIN}
logging: *default-logging
networks:
- weatherapi
- google-sheets

networks:
weatherapi:
name: weatherapi
google-sheets:
name: google-sheets
3 changes: 3 additions & 0 deletions google_sheets/__about__.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
"""A simple Google Sheets API made for usage with FastAgency."""

__version__ = "0.1.0"
5 changes: 5 additions & 0 deletions google_sheets/__init__.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
"""A simple Google Sheets API made for usage with FastAgency."""

from .__about__ import __version__

__all__ = ["__version__"]
2 changes: 1 addition & 1 deletion weatherapi/app.py → google_sheets/app.py
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
app = FastAPI(
servers=[{"url": base_url, "description": "Weather app server"}],
version=__version__,
title="WeatherAPI",
title="google-sheets",
)


Expand Down
28 changes: 14 additions & 14 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,8 @@ requires = ["hatchling"]
build-backend = "hatchling.build"

[project]
name = "weatherapi"
description = "WeatherAPI: A simple weather API made for usage with FastAgency"
name = "google-sheets"
description = "google-sheets: A simple Google Sheets API made for usage with FastAgency"
readme = "README.md"
authors = [
{ name = "airt", email = "[email protected]" },
Expand Down Expand Up @@ -78,26 +78,26 @@ test-core = [
]

testing = [
"weatherapi[test-core]",
"weatherapi[server]", # Uvicorn is needed for testing
"google-sheets[test-core]",
"google-sheets[server]", # Uvicorn is needed for testing
]

dev = [
"weatherapi[server,lint,testing]",
"google-sheets[server,lint,testing]",
"pre-commit==3.7.1",
"detect-secrets==1.5.0",
]

[project.urls]
Tracker = "https://github.com/airtai/weatherapi/issues"
Source = "https://github.com/airtai/weatherapi"
Tracker = "https://github.com/airtai/google-sheets/issues"
Source = "https://github.com/airtai/google-sheets"
Discord = "https://discord.gg/qFm6aSqq59"

[project.scripts]
# weatherapi = "weatherapi.__main__:cli"
# google_sheets = "google_sheets.__main__:cli"

[tool.hatch.version]
path = "weatherapi/__about__.py"
path = "google_sheets/__about__.py"

[tool.hatch.build]
skip-excluded-dirs = true
Expand All @@ -110,15 +110,15 @@ exclude = [
allow-direct-references = true

[tool.hatch.build.targets.wheel]
only-include = ["weatherapi"]
only-include = ["google_sheets"]

[tool.hatch.build.targets.wheel.sources]
"src" = ""
# "scripts" = "weatherapi/templates"
# "scripts" = "google_sheets/templates"

[tool.mypy]

files = ["weatherapi", "tests"]
files = ["google_sheets", "tests"]

strict = true
python_version = "3.9"
Expand All @@ -145,7 +145,7 @@ disallow_any_unimported = false
fix = true
line-length = 88
# target-version = 'py39'
include = ["weatherapi/**/*.py", "weatherapi/**/*.pyi", "tests/**/*.py", "pyproject.toml"]
include = ["google_sheets/**/*.py", "google_sheets/**/*.pyi", "tests/**/*.py", "pyproject.toml"]
exclude = ["docs/docs_src"]

[tool.ruff.lint]
Expand Down Expand Up @@ -206,7 +206,7 @@ concurrency = [
"thread"
]
source = [
"weatherapi",
"google_sheets",
# "tests",
]
context = '${CONTEXT}'
Expand Down
2 changes: 1 addition & 1 deletion scripts/deploy.sh
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ fi

ssh_command="ssh -o StrictHostKeyChecking=no -i key.pem azureuser@$DOMAIN"

container_name="weatherapi"
container_name="google-sheets"
log_file="${container_name}.log"

echo "INFO: Capturing docker container logs"
Expand Down
2 changes: 1 addition & 1 deletion scripts/lint-pre-commit.sh
Original file line number Diff line number Diff line change
Expand Up @@ -28,5 +28,5 @@ pip install --editable ".[dev]" \
# and specify the package to run on explicitly.
# Note that we do not use --ignore-missing-imports,
# as this can give us false confidence in our results.
# mypy weatherapi
# mypy google-sheets
./scripts/lint.sh
4 changes: 2 additions & 2 deletions scripts/lint.sh
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
#!/usr/bin/env bash

echo "Running pyup_dirs..."
pyup_dirs --py38-plus --recursive weatherapi tests
pyup_dirs --py38-plus --recursive google_sheets tests

echo "Running ruff linter (isort, flake, pyupgrade, etc. replacement)..."
ruff check
Expand All @@ -10,4 +10,4 @@ echo "Running ruff formater (black replacement)..."
ruff format

# echo "Running black..."
# black weatherapi examples tests docs
# black google_sheets examples tests docs
2 changes: 1 addition & 1 deletion scripts/run_server.sh
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
#!/usr/bin/env bash

uvicorn weatherapi.app:app --workers 2 --host 0.0.0.0 --proxy-headers
uvicorn google_sheets.app:app --workers 2 --host 0.0.0.0 --proxy-headers
2 changes: 1 addition & 1 deletion scripts/static-analysis.sh
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ echo "Running mypy..."
mypy

echo "Running bandit..."
bandit -c pyproject.toml -r weatherapi
bandit -c pyproject.toml -r google_sheets

echo "Running semgrep..."
semgrep scan --config auto --error
2 changes: 1 addition & 1 deletion scripts/static-pre-commit.sh
Original file line number Diff line number Diff line change
Expand Up @@ -28,5 +28,5 @@ pip install --editable ".[dev]" \
# and specify the package to run on explicitly.
# Note that we do not use --ignore-missing-imports,
# as this can give us false confidence in our results.
# mypy weatherapi
# mypy google-sheets
./scripts/static-analysis.sh
6 changes: 3 additions & 3 deletions tests/app/test_app.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@

from fastapi.testclient import TestClient

from weatherapi import __version__ as version
from weatherapi.app import app
from google_sheets import __version__ as version
from google_sheets.app import app

client = TestClient(app)

Expand Down Expand Up @@ -37,7 +37,7 @@ def test_weather_route(self) -> None:
def test_openapi(self) -> None:
expected = {
"openapi": "3.1.0",
"info": {"title": "WeatherAPI", "version": version},
"info": {"title": "google-sheets", "version": version},
"servers": [
{"url": "http://localhost:8000", "description": "Weather app server"}
],
Expand Down
3 changes: 0 additions & 3 deletions weatherapi/__about__.py

This file was deleted.

5 changes: 0 additions & 5 deletions weatherapi/__init__.py

This file was deleted.

Loading