Skip to content
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

Calculate new repository link from ssh path in configure script #1614

Merged
merged 3 commits into from
Oct 30, 2024

Conversation

gazayas
Copy link
Contributor

@gazayas gazayas commented Jul 16, 2024

Closes #1607.

I figured we didn't want to entirely get rid of the ask methods here in case developers decide not to setup their project with GitHub. I tested the configure script against a personal repository of mine so it should work, but you can pull it down with this branch if you want to check it out:

> git clone -b refactor-github-details [email protected]:bullet-train-co/bullet_train.git bt_test

@jagthedrummer
Copy link
Contributor

Closing in favor of #1727

Copy link
Contributor

@jagthedrummer jagthedrummer left a comment

Choose a reason for hiding this comment

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

Thanks, @gazayas, this looks great! (Sorry it took me so long to get to it. 😬 )

@jagthedrummer jagthedrummer merged commit d34a693 into main Oct 30, 2024
42 checks passed
@jagthedrummer jagthedrummer deleted the refactor-github-details branch October 30, 2024 16:45
@gazayas
Copy link
Contributor Author

gazayas commented Oct 30, 2024

@jagthedrummer No problem, thanks for giving it a look!

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.

bin/configure tasks question it doesn't need to
2 participants