Don't show warning for `"past end of"`

[pokey]

This behaviour could actually be useful in rare cases.

• Remove warning
• Add note to old videos that use "past end of" to use "tail"

[AndreasArvidsson]

@pokey I was just about to implement this one, but I can't do the second task so it's probably better if you take ownership? Or do you want me to go ahead and do the first task and then hand it over to you?

This is where in the code the warning is defined

cursorless/packages/cursorless-engine/src/core/commandVersionUpgrades/canonicalizeAndValidateCommand.ts

Lines 122 to 151 in 0e4c9d9

	export function checkForOldInference(
	partialTargets: PartialTargetDescriptor[],
	) {
	const hasOldInference = partialTargets.some((target) => {
	return (
	target.type === "range" &&
	target.active.mark == null &&
	target.active.modifiers?.some((m) => m.type === "position") &&
	!target.active.modifiers?.some((m) => m.type === "inferPreviousMark")
	);
	});
	if (hasOldInference) {
	const { globalState, messages } = ide();
	const hideInferenceWarning = globalState.get("hideInferenceWarning");
	if (!hideInferenceWarning) {
	showWarning(
	messages,
	"deprecatedPositionInference",
	'The "past start of" / "past end of" form has changed behavior. For the old behavior, update cursorless-talon (https://www.cursorless.org/docs/user/updating/), and then you can now say "past start of its" / "past end of its". For example, "take air past end of its line". You may also consider using "head" / "tail" instead; see https://www.cursorless.org/docs/#head-and-tail',
	"Don't show again",
	).then((pressed) => {
	if (pressed) {
	globalState.set("hideInferenceWarning", true);
	}
	});
	}
	}
	}

[pokey]

I don't feel strongly either way. I think either is fine

[AndreasArvidsson]

@pokey The warning is now removed and I have assigned it to us you can update your videos.