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

Add an <internet/> kind to requires/recommends/supports (2) #425

Merged
merged 2 commits into from Aug 22, 2022

Conversation

ximion
Copy link
Owner

@ximion ximion commented Aug 20, 2022

This is a fixed up version of #421 that is ready to merge (due to @pwithnall being on vacation and there's no reason to leave this 99.9% completed feature out of the next release).

@cassidyjames @JakobDev Please have a look if you see any issues, otherwise I hope I can merge this in time for the release that will likely happen in the evening next Monday.

This allows apps to specify whether they require the internet to work,
or whether they explicitly work fully without internet.

Signed-off-by: Philip Withnall <pwithnall@endlessos.org>
Signed-off-by: Matthias Klumpp <matthias@tenstral.net>

Fixes: #343
@JakobDev
Copy link
Contributor

It is missing a validation, but otherwise it looks OK

@ximion
Copy link
Owner Author

ximion commented Aug 22, 2022

Indeed! I'll add validation for it before this is released.

@ximion ximion merged commit d7ebcf9 into master Aug 22, 2022
16 checks passed
@ximion ximion deleted the wip/mak/rel_internet branch August 22, 2022 10:07
@pwithnall
Copy link
Contributor

Yay, thanks everyone :)

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.

None yet

3 participants