Skip to content

Removing dead/unused code on the branch - #1209

Open
EngCaioFonseca wants to merge 1 commit into
devfrom
fix/cache-empty-repolist-error-handling
Open

EngCaioFonseca wants to merge 1 commit into
devfrom
fix/cache-empty-repolist-error-handling

Conversation

@EngCaioFonseca

@EngCaioFonseca EngCaioFonseca commented Sep 4, 2026 •

Copy link
Copy Markdown
Contributor

Pull Request Change Description

Fixes four issues in PostgreSQL data-access and startup error handling. The changes are local and introduce no schema, dependency, or callback-graph changes.

The PR changed scope to remove the dead/unused code found in this branch.

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? Claude Opus 5, GPT 5.6 - sol, GPT 6 - Astra.
- How were these tools used? Draft, multiple iterations, review.
- Did you review these outputs before submitting this PR? Yes.

@EngCaioFonseca EngCaioFonseca self-assigned this Sep 4, 2026
@EngCaioFonseca EngCaioFonseca added the bug Something isn't working label Sep 4, 2026
@EngCaioFonseca EngCaioFonseca moved this from Backlog to Testing | Waiting on PR response | Blocked in Aspen Project Board Sep 4, 2026
@EngCaioFonseca
EngCaioFonseca force-pushed the fix/cache-empty-repolist-error-handling branch from 584a066 to 1b18321 Compare September 14, 2026 22:25
@EngCaioFonseca
EngCaioFonseca marked this pull request as ready for review September 14, 2026 22:35
@cdolfi

cdolfi commented Sep 17, 2026

Copy link
Copy Markdown
Collaborator

Do we have any examples of these errors happening? What prompted the making of this PR?

@cdolfi

cdolfi commented Sep 30, 2026

Copy link
Copy Markdown
Collaborator

Discussed during team meeting that this would be updated to take out legacy used code rather than add error handling to it

@EngCaioFonseca

Copy link
Copy Markdown
Contributor Author

Just needs to be modified to removing dead/unused code.

@EngCaioFonseca
EngCaioFonseca force-pushed the fix/cache-empty-repolist-error-handling branch 2 times, most recently from 3fdb7d8 to 744f11d Compare October 13, 2026 12:49
@EngCaioFonseca EngCaioFonseca changed the title Fix/cache empty repolist error handling Removing dead/unused code on the branch Oct 13, 2026
@EngCaioFonseca EngCaioFonseca removed the bug Something isn't working label Oct 13, 2026
Signed-off-by: Caio Fonseca <engcaiofonseca@protonmail.com>
@EngCaioFonseca
EngCaioFonseca force-pushed the fix/cache-empty-repolist-error-handling branch from 744f11d to 4125b47 Compare October 13, 2026 13:08
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.

2 participants