Skip to content

Conversation

riffy
Copy link
Contributor

@riffy riffy commented Jun 4, 2025

Based on request #261 , a new setting (with section "Environment" ) is introduced.
When the clone repository dialog is opened, the settings is read.
The clonerepository dialog contains a checkbox to allow overwriting the setting from the dialog.

grafik

grafik

Since this is my first touch with Kotlin, please always ELI5 :)

@riffy
Copy link
Contributor Author

riffy commented Jun 18, 2025

Hey @JetpackDuba , can i have a review ? Want to know if I could improve something here (really like gitnuro and want to improve it)

@JetpackDuba
Copy link
Owner

JetpackDuba commented Jun 18, 2025

Hello!

Yes, sorry. I have been a bit busy with my job and some health issues. Probably in a week or so I will be back to normal and review everything pending.

Sorry again for the delay.

@riffy
Copy link
Contributor Author

riffy commented Aug 9, 2025

Sorry, I forgot about this PR :/ I've made the respective changes, so have another look @JetpackDuba .

Repository owner deleted a comment from Anusorn1997 Aug 9, 2025
Repository owner deleted a comment from Anusorn1997 Aug 9, 2025
@JetpackDuba
Copy link
Owner

Hello!
That looks good :D Could you please rebase main before merging?

@riffy
Copy link
Contributor Author

riffy commented Aug 12, 2025

Hi! Done :) Feel free to merge

@JetpackDuba JetpackDuba merged commit bc1c733 into JetpackDuba:main Aug 12, 2025
1 check failed
@JetpackDuba
Copy link
Owner

I've squashed it into a single commit since the other commits were small and I didn't want to have to merges from main to the branch displayed.

Thanks!

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.

3 participants