Repository: Khan/genqlient
Pull request #246 (2022-12-22 14:37:23)

PR 239 update: Adding more logging for the initial generate setup
This is just a tweak to #239 to update the snapshots.
I have:
- [X] Written a clear PR title and description (above)
- [X] Signed the [Khan Academy CLA](https://www.khanacademy.org/r/cla)
- [X] Added tests covering my changes, if applicable
- [X] Included a link to the issue fixed, if applicable
- [X] Included documentation, for new features
- [ ] Added an entry to the changelog


Patch:
diff --git a/generate/config.go b/generate/config.go
index 7b98d2a..3881c97 100644
--- a/generate/config.go
+++ b/generate/config.go
@@ -102,12 +102,16 @@ func (c *Config) ValidateAndFillDefaults(baseDir string) error {
 	if c.Package == "" {
 		abs, err := filepath.Abs(c.Generated)
 		if err != nil {
-			return errorf(nil, "unable to guess package-name: %v", err)
+			return errorf(nil, "unable to guess package-name: %v is not a valid identifier"+
+				"\nSet package name in genqlient.yaml"+
+				"\nExample: https://github.com/Khan/genqlient/blob/main/example/genqlient.yaml#L6", err)
 		}
 
 		base := filepath.Base(filepath.Dir(abs))
 		if !token.IsIdentifier(base) {
-			return errorf(nil, "unable to guess package-name: %v is not a valid identifier", base)
+			return errorf(nil, "unable to guess package-name: %v is not a valid identifier"+
+				"\nSet package name in genqlient.yaml"+
+				"\nExample: https://github.com/Khan/genqlient/blob/main/example/genqlient.yaml#L6", base)
 		}
 
 		c.Package = base
diff --git a/generate/convert.go b/generate/convert.go
index 8073ae5..caf3754 100644
--- a/generate/convert.go
+++ b/generate/convert.go
@@ -524,7 +524,8 @@ func (g *generator) convertDefinition(
 
 		// (If you had an entry in bindings, we would have returned it above.)
 		return nil, errorf(
-			pos, `unknown scalar %v: please add it to "bindings" in genqlient.yaml`, def.Name)
+			pos, "unknown scalar %v: please add it to \"bindings\" in genqlient.yaml"+
+				"\nExample: https://github.com/Khan/genqlient/blob/main/example/genqlient.yaml#L12", def.Name)
 	default:
 		return nil, errorf(pos, "unexpected kind: %v", def.Kind)
 	}


Test patch:
diff --git a/generate/testdata/snapshots/TestGenerateErrors-UnknownScalar-go b/generate/testdata/snapshots/TestGenerateErrors-UnknownScalar-go
index 3c698d2..ec3378e 100644
--- a/generate/testdata/snapshots/TestGenerateErrors-UnknownScalar-go
+++ b/generate/testdata/snapshots/TestGenerateErrors-UnknownScalar-go
@@ -1,1 +1,2 @@
 testdata/errors/UnknownScalar.schema.graphql:3: unknown scalar UnknownScalar: please add it to "bindings" in genqlient.yaml
+Example: https://github.com/Khan/genqlient/blob/main/example/genqlient.yaml#L12
diff --git a/generate/testdata/snapshots/TestGenerateErrors-UnknownScalar-graphql b/generate/testdata/snapshots/TestGenerateErrors-UnknownScalar-graphql
index 3c698d2..ec3378e 100644
--- a/generate/testdata/snapshots/TestGenerateErrors-UnknownScalar-graphql
+++ b/generate/testdata/snapshots/TestGenerateErrors-UnknownScalar-graphql
@@ -1,1 +1,2 @@
 testdata/errors/UnknownScalar.schema.graphql:3: unknown scalar UnknownScalar: please add it to "bindings" in genqlient.yaml
+Example: https://github.com/Khan/genqlient/blob/main/example/genqlient.yaml#L12