-
Notifications
You must be signed in to change notification settings - Fork 1
merge walker #13
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
base: main
Are you sure you want to change the base?
merge walker #13
Conversation
--- For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/JohnDaWalka/test-aoc?shareId=XXXX-XXXX-XXXX-XXXX).
Integrate code into GitHub project
Reviewer's Guide by SourceryThis pull request updates the repository link in the README file to point to the correct repository, ensuring that users are directed to the appropriate location for information and contributions. No diagrams generated as the changes look simple and do not need a visual representation. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @JohnDaWalka - I've reviewed your changes - here's some feedback:
Overall Comments:
- Please ensure the pull request title is descriptive.
- The description should explain the purpose of the pull request.
Here's what I looked at during the review
- 🟡 General issues: 1 issue found
- 🟢 Security: all looks good
- 🟢 Review instructions: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
mergin repos
* **create-puzzle-issue.yml** - Add a step to use `actions/checkout@v4` before creating issues - Update `GH_TOKEN` to use `${{ secrets.GH_TOKEN }}` * **merge-sourcery-ai-bot-pr.yml** - Add a step to use `actions/checkout@v4` before merging pull requests - Update `GH_TOKEN` to use `${{ secrets.GH_TOKEN }}` * **submit-advent-of-code.yml** - Update `GH_TOKEN` to use `${{ secrets.GH_TOKEN }}`
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
code integration
…kflows * **create-puzzle-issue.yml** - Add a step to merge `develop` into `main` after creating the puzzle issue - Use `gh pr merge` command with `--squash`, `--auto`, and `--delete-branch` options * **merge-sourcery-ai-bot-pr.yml** - Add a step to merge `develop` into `main` after merging the pull request - Use `gh pr merge` command with `--squash`, `--auto`, and `--delete-branch` options * **submit-advent-of-code.yml** - Add a step to merge `develop` into `main` after submitting the solution - Use `gh pr merge` command with `--squash`, `--auto`, and `--delete-branch` options * **README.md** - Update documentation to include information about new GitHub Actions workflows for merging code - Provide examples of how to use `gh pr merge` command with `--squash`, `--auto`, and `--delete-branch` options
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
using sorcery for poker coach
* **poker_coach/ui.py** - Create a new file to implement the interactive UI for the poker coach using Tkinter - Implement the necessary code to create the interactive UI with hand entry and analysis functionality * **README.md** - Add a section on how to use the poker coach UI - Include installation and usage instructions for the poker coach UI * **tests/test_poker_coach_ui.py** - Create a new file to add tests for the poker coach UI - Write tests to ensure the UI functions correctly
Summary by Sourcery
Documentation: