-
-
Notifications
You must be signed in to change notification settings - Fork 444
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
snap: initial manifest and fixes #4091
base: main
Are you sure you want to change the base?
Conversation
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
I have read the CLA Document and I hereby sign the CLA |
46c2ec2
to
37577b9
Compare
37577b9
to
f65a746
Compare
@soumyaDghosh can you run How can we build it locally? it is enough to setup snapcraft and run the build command from the snap directory? |
Yup simple setup snapcraft and build from the project's root directory. |
(note that it's |
f65a746
to
4145994
Compare
I don't think the lint is failing due to my PR |
also |
The changes made in this PR also cause the lint to fail. We've had a brief issue where un-linted code got into the main branch, but this was resolved. Please
The issue you're referencing there is 3 years old, being resolved 2 years ago. |
4145994
to
d885245
Compare
This is the initial manifest. I have also created a patch to get the actual release details of the OS. If you want, I can also create a PR for the build and release.
Use the following Checklist if you have changed something on the Backend or Frontend: