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

add authors: [...] field to spago.yaml package.publish ? #1294

Open
srghma opened this issue Oct 11, 2024 · 2 comments · May be fixed by #1298
Open

add authors: [...] field to spago.yaml package.publish ? #1294

srghma opened this issue Oct 11, 2024 · 2 comments · May be fixed by #1298

Comments

@srghma
Copy link
Contributor

srghma commented Oct 11, 2024

some bower.json have authors field

{
  "name": "purescript-nullable",
  "authors": [
    "Phil Freeman <paf31@cantab.net>"
  ],
  "description": "A library for dealing with nulls",
  "license": "MIT",
  "repository": {
    "type": "git",
    "url": "https://github.com/purescript-contrib/purescript-nullable.git"
  },
  "ignore": [
    "**/.*",
    "bower_components",
    "node_modules",
    "output",
    "bower.json",
    "package.json"
  ]
}

mabye add this field to spago.yaml too?

package:
  name: nullable
  description: A library for dealing with nulls
  authors:
    - Phil Freeman <paf31@cantab.net>
Reading Spago workspace configuration...

✘ Couldn't parse Spago config, error:


  $.package: Unknown field(s): authors
@thomashoneyman
Copy link
Member

There is some potential overlap with the optional “owners” field, though that requires a public key if I remember correctly.

@f-f
Copy link
Member

f-f commented Oct 11, 2024

Yes, we have an upcoming (registry has it, spago is about to) owners key, which requires a public key. Is there a use case where the key is not desirable?

@f-f f-f linked a pull request Nov 4, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants