Skip to content

bookmarks: open Pull Request on GitHub from right-click menu#24

Merged
hewigovens merged 1 commit intomainfrom
test-pr-menu
Apr 28, 2026
Merged

bookmarks: open Pull Request on GitHub from right-click menu#24
hewigovens merged 1 commit intomainfrom
test-pr-menu

Conversation

@hewigovens
Copy link
Copy Markdown
Owner

No description provided.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces the ability to open GitHub Pull Request URLs directly from the application. It adds logic to the core library for extracting repository slugs from various Git remote formats and generating PR compose URLs, which are then exposed to the macOS shell. The UI is updated to include 'Pull Request on GitHub' buttons in the bookmark manager and DAG context menus. Feedback focuses on ensuring bookmark names are URL-encoded to prevent malformed links and refining the error message shown when a GitHub remote cannot be determined.

Comment thread crates/jayjay-core/src/repo/github.rs Outdated
Comment thread shell/mac/Sources/JayJay/Repo/ViewModel/Actions/RepoViewModel+GitActions.swift Outdated
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f43acd7be2

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread crates/jayjay-core/src/repo/github.rs
@hewigovens hewigovens merged commit e8b4826 into main Apr 28, 2026
2 checks passed
@hewigovens hewigovens deleted the test-pr-menu branch April 28, 2026 12: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.

1 participant