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

Add make sprint-data-import and issue-data-import to import github sprint and issue data to database #84

Merged
merged 14 commits into from
Jun 26, 2024
Merged
8 changes: 4 additions & 4 deletions .grype.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,11 +16,11 @@ ignore:
- fix-state: not-fixed
- fix-state: wont-fix
- fix-state: unknown
# Golang vulnerability inside of a python docker image. It's basically impossible to find
# out where the impacted golang vulnerability is coming from. The python image in question
# does not even have golang installed, yet somehow still there's a golang vulnerability.
# We are ignoring the finding because it would take undue effort to track down the source.
# Golang vulnerabilities inside of a python docker image. Both originate from lower level packages within the GitHub CLI:
# https://github.com/cli/cli/blob/trunk/go.mod#L101
# https://github.com/cli/cli/blob/trunk/go.mod#L161
- vulnerability: GHSA-4v7x-pqxf-cx7m
- vulnerability: GHSA-v6v8-xj6m-xwqh
# https://github.com/anchore/grype/issues/1172
- vulnerability: GHSA-xqr8-7jwr-rhp7
- vulnerability: GHSA-7fh5-64p2-3v2j
Expand Down
9 changes: 9 additions & 0 deletions analytics/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -146,6 +146,15 @@ sprint-data-export:
--project $(SPRINT_PROJECT) \
--output-file $(SPRINT_FILE)

# Import file data to database
# Takes sprint and issue and puts it in database
sprint-db-data-import:
@echo "=> Importing project data to the database"
@echo "====================================================="
$(POETRY) analytics export db_export \
Copy link
Collaborator

Choose a reason for hiding this comment

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

⭐⭐⭐   (must change before approval) The makefile command says db-import whereas the python command is db-export. You'll want to pick one or the other. I'm thinking import?

Copy link
Author

Choose a reason for hiding this comment

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

I'm also thinking import

Copy link
Member

Choose a reason for hiding this comment

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

Please update with import. Please also create a separate import app:

import_app = typer.Typer()
...
app.add_typer(import_app, name="import", help="Import data into the database")

--sprint-file $(SPRINT_FILE) \
--issue-file $(ISSUE_FILE)

roadmap-data-export:
@echo "=> Exporting project data from the product roadmap"
@echo "====================================================="
Expand Down
Loading
Loading