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

cmd/cue: cmd fails to report errors in task names #1461

Open
vikstrous2 opened this issue Jan 12, 2022 · 2 comments
Open

cmd/cue: cmd fails to report errors in task names #1461

vikstrous2 opened this issue Jan 12, 2022 · 2 comments

Comments

@vikstrous2
Copy link

What version of CUE are you using (cue version)?

$ cue version
cue version v0.4.1 linux/amd64

Does this issue reproduce with the latest release?

yes

What did you do?

cat > test_tool.cue
package test

import (
	"tool/cli"
)

command: {
	write: task: {
		"\(*"a"|*"b")": cli.Print & {
		  text: "TEST1"
		}
	}
}
cue cmd  write ./...

What did you expect to see?

an error about the disjunction not being valid

What did you see instead?

no output, silent failure

@vikstrous2 vikstrous2 added NeedsInvestigation Triage Requires triage/attention labels Jan 12, 2022
@vikstrous2 vikstrous2 changed the title _tool .cue fails to report errors in command names _tool.cue fails to report errors in task names Jan 12, 2022
@myitcv myitcv added NeedsDecision tooling and removed NeedsInvestigation Triage Requires triage/attention labels Jan 13, 2022
@myitcv myitcv changed the title _tool.cue fails to report errors in task names cmd/cue: cmd fails to report errors in task names Jan 13, 2022
@myitcv
Copy link
Member

myitcv commented Jan 13, 2022

Confirmed with d4d0f14.

This has overlap with our plans for cuerun insofar as the method of declaring commands might change. Nonetheless, regardless of how commands are specified, we should better surface errors.

@mpvl
Copy link
Member

mpvl commented Jan 15, 2022

This is independent of the decision about cuerun.

@myitcv myitcv added the zGarden label Jun 15, 2023
@mvdan mvdan removed the zGarden label Feb 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants