Skip to content

Refactor project structure and improve CLI command handling#70

Merged
chrisreddington merged 5 commits intomainfrom
restructure
Feb 10, 2025
Merged

Refactor project structure and improve CLI command handling#70
chrisreddington merged 5 commits intomainfrom
restructure

Conversation

@chrisreddington
Copy link
Collaborator

Restructure the project by moving packages to an internal folder, refactoring the main application into distinct packages, and enhancing command handling for the CLI tool. Update the GitHub profile URL to use the default hostname.

Copilot AI review requested due to automatic review settings January 29, 2025 12:22
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 42 out of 42 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

cmd/skyline/skyline.go:59

  • This error path is not covered by tests. Add a test scenario that mocks fetchContributionData to return an error so that this code path is exercised.
return err

Tip: Copilot only keeps its highest confidence comments to reduce noise and keep you focused. Learn more

@chrisreddington chrisreddington merged commit e6d1541 into main Feb 10, 2025
17 checks passed
@chrisreddington chrisreddington deleted the restructure branch February 10, 2025 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants