Skip to content

renamed base to base_url (fixes #1607)#1629

Merged
mre merged 8 commits intolycheeverse:masterfrom
Arteiii:master
Feb 16, 2025
Merged

renamed base to base_url (fixes #1607)#1629
mre merged 8 commits intolycheeverse:masterfrom
Arteiii:master

Conversation

@Arteiii
Copy link
Contributor

@Arteiii Arteiii commented Feb 10, 2025

Changed the configuration parameter base to base_url to improve clarity (fixes #1607)

Arteiii and others added 2 commits February 14, 2025 14:25
Co-authored-by: Matthias Endler <matthias@endler.dev>
Co-authored-by: Matthias Endler <matthias@endler.dev>
@mre
Copy link
Member

mre commented Feb 14, 2025

Just one final remark; rest of the code looks fine.

@Arteiii
Copy link
Contributor Author

Arteiii commented Feb 14, 2025

well I actually don't know why this test fails...

@mre
Copy link
Member

mre commented Feb 14, 2025

I think it's just a network blip. Let's retry!

@mre
Copy link
Member

mre commented Feb 14, 2025

Hum, no. The example.com website is up, so no clue. It's unrelated to your changes, but it would be nice to get that green tick. I can merge it if you like or we try again later.

@Arteiii
Copy link
Contributor Author

Arteiii commented Feb 15, 2025

Haha idk, weird i can Check tomorrow

@mre mre merged commit d6bbf85 into lycheeverse:master Feb 16, 2025
6 checks passed
@mre mre mentioned this pull request Feb 16, 2025
@mre
Copy link
Member

mre commented Feb 16, 2025

It was just a network issue. Now fixed. Thanks for the pull request! 😃

@mre mre mentioned this pull request Feb 24, 2025
@mre mre mentioned this pull request May 13, 2025
@mre mre mentioned this pull request May 25, 2025
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.

Rename --base to --base-url

2 participants