fix: snapcraft core22 and core24 support [#8548]

[filfreire]

Closes #8548

Not tested yet

cc @mmaietta

[changeset-bot]

🦋 Changeset detected

Latest commit: 2b5c6ed

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 8 packages

Name	Type
app-builder-lib	Patch
dmg-builder	Patch
electron-builder-squirrel-windows	Patch
electron-builder	Patch
electron-forge-maker-appimage	Patch
electron-forge-maker-nsis-web	Patch
electron-forge-maker-nsis	Patch
electron-forge-maker-snap	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[netlify]

✅ Deploy Preview for car-park-attendant-cleat-11576 ready!

Name	Link
🔨 Latest commit	2b5c6ed
🔍 Latest deploy log	https://app.netlify.com/sites/car-park-attendant-cleat-11576/deploys/66fc15aabfca460008cbb623
😎 Deploy Preview	https://deploy-preview-8549--car-park-attendant-cleat-11576.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[mmaietta]

Thanks for your contribution! We'll also want a new unit test for core24.

You can probably just copy-paste this test and update the snap.base value

electron-builder/test/src/linux/snapTest.ts

Lines 314 to 330 in fc3a78e

	test.ifDevOrLinuxCi(
	"base option",
	app({
	targets: snapTarget,
	config: {
	productName: "Sep",
	snap: {
	base: "core22",
	},
	},
	effectiveOptionComputed: async ({ snap }) => {
	expect(snap).toMatchSnapshot()
	expect(snap.base).toBe("core22")
	return true
	},
	})
	)

Why is this considered a major semver bump though? Seems more like a minor or even patch semver for fixing core22 and a new feature for core24 as I don't see any breaking API changes

[filfreire]

@mmaietta added a test for base core24 and edited the changeset to something lower than major

[mmaietta]

@filfreire is this tested yet since it's marked ready for review? If so, please update the PR description accordingly when you have a chance.