Skip to content

Re enable repo info query - #1199

Merged
cdolfi merged 6 commits into
devfrom
re-enable-repo-info
Sep 16, 2026
Merged

cdolfi merged 6 commits into
devfrom
re-enable-repo-info

Conversation

@EngCaioFonseca

@EngCaioFonseca EngCaioFonseca commented Jul 27, 2026 •

Copy link
Copy Markdown
Contributor

This pr re-enable the repo general info query by uncommenting it and making adaptations.

Pull Request Change Description

Generative AI disclosure

Please select one option:

  • This contribution was NOT assisted or created by Generative AI tools.
  • This contribution was assisted or created by Generative AI tools.

If AI tools were used, please provide details below:
- What tools were used?
- How were these tools used?
- Did you review these outputs before submitting this PR?

@EngCaioFonseca EngCaioFonseca self-assigned this Jul 27, 2026
@EngCaioFonseca EngCaioFonseca moved this from Backlog to Testing | Waiting on PR response | Blocked in Aspen Project Board Jul 27, 2026
Signed-off-by: Caio Fonseca <engcaiofonseca@protonmail.com>
Signed-off-by: Caio Fonseca <engcaiofonseca@protonmail.com>
@EngCaioFonseca
EngCaioFonseca marked this pull request as ready for review July 27, 2026 20:52
@EngCaioFonseca
EngCaioFonseca requested a review from MoralCode July 27, 2026 20:52
Comment thread 8Knot/pages/repo_overview/visualizations/repo_general_info.py
Signed-off-by: Caio Fonseca <engcaiofonseca@protonmail.com>
@EngCaioFonseca

Copy link
Copy Markdown
Contributor Author

This should be ready to go

Comment thread 8Knot/pages/repo_overview/visualizations/repo_general_info.py Outdated
Signed-off-by: Caio Fonseca <engcaiofonseca@protonmail.com>
Signed-off-by: Caio Fonseca <engcaiofonseca@protonmail.com>
Signed-off-by: Caio Fonseca <engcaiofonseca@protonmail.com>

@cdolfi cdolfi left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm! tested locally and worked

@MoralCode MoralCode left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

my concerns about improperly uncommented stuff seem to be resolved

leaving a generic comment review to see if that clears my previous "requested changes"

@MoralCode MoralCode left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

apparrently that did not work, heres an approve (relying on calis assertion that it is tested and works)

@cdolfi
cdolfi merged commit 3828958 into dev Sep 16, 2026
7 checks passed
@github-project-automation github-project-automation Bot moved this from Testing | Waiting on PR response | Blocked to Sprint 55 in Aspen Project Board Sep 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Re enable repo general info that is based on the repo files query

3 participants