Skip to content

1.0.0 release#2

Merged
tehlordvortex merged 1 commit intomainfrom
05-08-1.0.0_release
May 9, 2025
Merged

1.0.0 release#2
tehlordvortex merged 1 commit intomainfrom
05-08-1.0.0_release

Conversation

@tehlordvortex
Copy link
Copy Markdown
Contributor

@tehlordvortex tehlordvortex commented May 8, 2025 •
edited
Loading

release v1 of the mcp-cli

@rover-app
Copy link
Copy Markdown

rover-app bot commented May 8, 2025

Hey, intrepid hero!

We're scanning your PR for issues. Stand by for comments.

That's all, folks!
Rover 🤖

@tehlordvortex tehlordvortex marked this pull request as ready for review May 8, 2025 18:07
@tehlordvortex Graphite App
Copy link
Copy Markdown
Contributor Author

tehlordvortex commented May 8, 2025 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

@rover-app
Copy link
Copy Markdown

rover-app bot commented May 8, 2025

Rover alert scan for Rover
Scanned at 06:07PM May 8th 2025 for commit 6b6ae72dde325a29b4e05a51dc3b63e1ff68b9cf

Alert generation for this PR was skipped for the following reason:

No file changes to check.

@tehlordvortex tehlordvortex marked this pull request as draft May 8, 2025 18:08
@tehlordvortex tehlordvortex force-pushed the 05-08-1.0.0_release branch from 6b6ae72 to fa8c958 Compare May 8, 2025 18:09
@tehlordvortex tehlordvortex force-pushed the 05-08-1.0.0_release branch from fa8c958 to c053c1b Compare May 9, 2025 08:05
isaacharrisholt
isaacharrisholt reviewed May 9, 2025
View reviewed changes
README.md
@@ -16,3 +16,6 @@

Copy link
Copy Markdown
Contributor

@isaacharrisholt isaacharrisholt May 9, 2025

Choose a reason for hiding this comment

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

[Re: line +10]

Might be worth explicitly having something like:

        "ROVER_API_KEY": "YOUR_API_KEY"

See this comment inline on Graphite.

README.md
@@ -16,3 +16,6 @@

Copy link
Copy Markdown
Contributor

@isaacharrisholt isaacharrisholt May 9, 2025

Choose a reason for hiding this comment

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

[Re: lines +7 to +8]

The bunx command doesn't have a -y flag, and you'd probably want to run it with bunx --bun @getrover/mcp-cli@latest. Do you think we should have another example?

See this comment inline on Graphite.

Copy link
Copy Markdown
Contributor Author

@tehlordvortex tehlordvortex May 9, 2025

Choose a reason for hiding this comment

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

yeah, having another example works, i'll add a seperate bun example, ty!

@tehlordvortex tehlordvortex force-pushed the 05-08-1.0.0_release branch from c053c1b to 9530f0a Compare May 9, 2025 09:11
@tehlordvortex tehlordvortex marked this pull request as ready for review May 9, 2025 10:14
@tehlordvortex tehlordvortex force-pushed the 05-08-1.0.0_release branch from 9530f0a to a1afee3 Compare May 9, 2025 10:17
@tehlordvortex tehlordvortex merged commit f9dbeda into main May 9, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@TTitcombe TTitcombe Awaiting requested review from TTitcombe

1 more reviewer

@isaacharrisholt isaacharrisholt isaacharrisholt approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@tehlordvortex @isaacharrisholt