-
Notifications
You must be signed in to change notification settings - Fork 354
[Feature] Add Cursor as a developer tool #1152
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
Conversation
…wnload through browser, needs curl or wget)
|
I also fixed the issue with apt and dnf by just replacing 1.7 with latest in the latest commit. And verified it worked using my browser and wget |
|
also see shellcheck warnings |
Co-authored-by: adam <[email protected]>
Co-authored-by: adam <[email protected]>
Co-authored-by: adam <[email protected]>
Co-authored-by: adam <[email protected]>
I fixed all the shellcheck errors in the latest commit |
Co-authored-by: adam <[email protected]>
Co-authored-by: adam <[email protected]>
Co-authored-by: adam <[email protected]>
Co-authored-by: adam <[email protected]>
|
This is my first PR, so I apologize for all the mistakes. |
adamperkowski
left a comment
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.
some formatting stuff here. really hope this is tested
Co-authored-by: adam <[email protected]>
Co-authored-by: adam <[email protected]>
|
If it would be better to close this until I test it, I can do that |
|
I know for a fact the Arch install works just fine |
i drafted it. i know it's a lot of work, but i don't have a way of doing that for you right now, sorry. |
I will get on it. I appreciate your time |
ChrisTitusTech
left a comment
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.
Looks good!
Type of Change
Description
I have created a cursor.sh file in developer-tools, and added it to tab-data.toml with the correct entries.
Currently it doesn't get the latest version for Fedora and Debian/Ubuntu based systems. It uses a hard-coded URL for those.Testing
I did testing with the arch version. I curled the URL's for dnf and apt/nala, and it downloads the .deb file.
Impact
I hope it makes it easier for people to setup their Dev Environments because Cursor is very commonplace now.
Issues / other PR's related
There are no issues/PR''s related to this as far as I can tell, I did this out of self interest.
Additional Information
In the future, I would like it to auto pull the latest version of Cursor for dnf and apt.Checklist