Skip to content

Conversation

@kianmeng
Copy link

@kianmeng kianmeng commented Oct 8, 2022

Found via codespell -S po -L crate,homestate,trough

Copy link
Collaborator

@Diegovsky Diegovsky left a comment

Choose a reason for hiding this comment

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

looks good and simple enough

src/api/cache.rs Outdated
IgnoreExpiry, // always use cached value
Revalidate, // always query remote cache
IgnoreCached, // ignore cache alltogether
IgnoreCached, // ignore cache all together

Choose a reason for hiding this comment

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

Here, its actually altogether.

@Diegovsky
Copy link
Collaborator

i can't really merge this myself, but I can say the changes are needed and should be merged.

@xou816

@Diegovsky
Copy link
Collaborator

Could you please fix the merge conflicts? I don't think it could take much of your time.

I apologise for the 3 year wait.

@deusnovus
Copy link

With all due respect to xou816 and his awesome work, maybe it's a good time to hard fork this project... Not being able to merge anything, not even dependency bumps, has rendered the app completely unusable in 2025.

Found via `codespell -S po -L crate,homestate,trough`
@kianmeng kianmeng requested a review from Diegovsky June 3, 2025 15:15
@Diegovsky
Copy link
Collaborator

Diegovsky commented Jun 3, 2025

it's something me and @knokelmaat have talked about. I'm at a bit of crossroads in life right now, meaning I can and am able to passively maintain and review, however I wouldn't be able to give spot the attention it needs.

We definitely want to go the hard fork route. We have thought of a name also, but I wouldn't want to make the community suffer yet another irregular maintainer.

I also gave some time for @xou816 to see if they were coming back as they had some GH activity end of last year, but it seems they aren't able to commit still.

I'll keep you posted, hopefully I might open a new issue announcing this and part with a fork.

I appreciate the interest after all this time :)

@yioannides
Copy link

I hope it all works out, @Diegovsky ! I'm looking forward to help in any way I can (currently exploring GTK via Python, looking to expand to Rust as well / can also provide translations, UI mockups, vector assets etc)

@Diegovsky
Copy link
Collaborator

We will definitely need all the support you all can give! Thank you for your patience and love for the software for all this time!

As of today, I'm starting a new job and for a while will not be able to commit to anything serious while adjusting to the routine.

When I'm ready, I'll make sure to announce it.

@slosacoder
Copy link

Hey @Diegovsky

How is it looking right now?

I was also wondering about forking spot, because currently it looks largely unmaintained and there are still pretty crucial features missing and there is also some stuff i would like to implement, so if you are planning to make a fork, i would like to be apart of it

@slosacoder
Copy link

We will definitely need all the support you all can give! Thank you for your patience and love for the software for all this time!

As of today, I'm starting a new job and for a while will not be able to commit to anything serious while adjusting to the routine.

When I'm ready, I'll make sure to announce it.

Hey, sorry for pinging you again, but perhaps you missed my message, so im asking again, are you planning to fork this project? i would really be glad to help to maintan it / implement some of the missing features

@Diegovsky
Copy link
Collaborator

Diegovsky commented Aug 1, 2025

Sorry, I was pretty sure I had answered you! Github might have scrapped my response or something!

As of right now, I have limited bandwidth to code, but am willing to deal with organization aspects such as reviews, flathub, github org, etc.

Thankfully, college is on winter break until august 18, so I'm hopeful we can get a lot done until then. @knokelmaat are you able to join this new effort?

@Diegovsky Diegovsky changed the base branch from development to maintenance August 1, 2025 18:06
@Diegovsky Diegovsky requested review from Diegovsky and removed request for UnlimitedCookies August 1, 2025 18:07
@Diegovsky
Copy link
Collaborator

@UnlimitedCookies please accept the changes

Copy link

@UnlimitedCookies UnlimitedCookies left a comment

Choose a reason for hiding this comment

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

LGTM

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.

6 participants